Skip to content

Commit

Permalink
HBASE-26470 Use openlabtesting protoc on linux arm64 in HBASE 2.x (ap…
Browse files Browse the repository at this point in the history
…ache#3860)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
  • Loading branch information
markjens authored and stoty committed Dec 9, 2021
1 parent dd83f04 commit 74027e2
Showing 1 changed file with 16 additions and 1 deletion.
17 changes: 16 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -786,7 +786,7 @@
<artifactId>protobuf-maven-plugin</artifactId>
<version>${protobuf.plugin.version}</version>
<configuration>
<protocArtifact>com.google.protobuf:protoc:${external.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<protocArtifact>${external.protobuf.groupid}:protoc:${external.protoc.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/protobuf/</protoSourceRoot>
<clearOutputDirectory>false</clearOutputDirectory>
<checkStaleness>true</checkStaleness>
Expand Down Expand Up @@ -1464,7 +1464,9 @@
<log4j.version>1.2.17</log4j.version>
<mockito-core.version>2.28.2</mockito-core.version>
<!--Internally we use a different version of protobuf. See hbase-protocol-shaded-->
<external.protobuf.groupid>com.google.protobuf</external.protobuf.groupid>
<external.protobuf.version>2.5.0</external.protobuf.version>
<external.protoc.version>${external.protobuf.version}</external.protoc.version>
<protobuf.plugin.version>0.6.1</protobuf.plugin.version>
<thrift.path>thrift</thrift.path>
<thrift.version>0.14.1</thrift.version>
Expand Down Expand Up @@ -4032,6 +4034,19 @@
</pluginManagement>
</build>
</profile>
<profile>
<id>aarch64</id>
<properties>
<external.protobuf.groupid>org.openlabtesting.protobuf</external.protobuf.groupid>
<external.protoc.version>2.5.0.2</external.protoc.version>
</properties>
<activation>
<os>
<family>linux</family>
<arch>aarch64</arch>
</os>
</activation>
</profile>
</profiles>
<!-- See https://jira.codehaus.org/browse/MSITE-443 why the settings need to be here and not in pluginManagement. -->
<reporting>
Expand Down

0 comments on commit 74027e2

Please sign in to comment.