You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2021/12/17 13:14:24 UTC
[hbase-thirdparty] branch branch-3 updated: HBASE-26592 Fix the broken shaded protobuf module (#70)
This is an automated email from the ASF dual-hosted git repository.
zhangduo pushed a commit to branch branch-3
in repository https://gitbox.apache.org/repos/asf/hbase-thirdparty.git
The following commit(s) were added to refs/heads/branch-3 by this push:
new 0710f6b HBASE-26592 Fix the broken shaded protobuf module (#70)
0710f6b is described below
commit 0710f6b0d2b0ec8d5b2cc263ebd01b9626520d96
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Fri Dec 17 21:07:06 2021 +0800
HBASE-26592 Fix the broken shaded protobuf module (#70)
Signed-off-by: Pankaj Kumar <pa...@apache.org>
---
hbase-shaded-protobuf/pom.xml | 28 +++++++++++++++-------------
1 file changed, 15 insertions(+), 13 deletions(-)
diff --git a/hbase-shaded-protobuf/pom.xml b/hbase-shaded-protobuf/pom.xml
index cb0ac5d..3b46166 100644
--- a/hbase-shaded-protobuf/pom.xml
+++ b/hbase-shaded-protobuf/pom.xml
@@ -32,6 +32,15 @@
Pulls down protobuf, patches it, compiles, and then relocates/shades.
</description>
<build>
+ <resources>
+ <resource>
+ <directory>src/main/java</directory>
+ <includes>
+ <!-- include the protos -->
+ <include>google/**</include>
+ </includes>
+ </resource>
+ </resources>
<plugins>
<plugin>
<!--Clean needs to purge src/main/java since this is where
@@ -78,6 +87,12 @@
<goal>unpack</goal>
</goals>
<configuration>
+ <!--
+ The protobuf-java has no compile or runtime dependency so we
+ only depend it here, without adding it to our dependencies
+ section. Need to review later if protobuf-java has compile or
+ runtime dependencies in the future.
+ -->
<artifactItems>
<artifactItem>
<groupId>com.google.protobuf</groupId>
@@ -148,12 +163,6 @@
<configuration>
<shadeSourcesContent>true</shadeSourcesContent>
<createSourcesJar>true</createSourcesJar>
- <artifactSet>
- <excludes>
- <!-- We will compile our own version, so exclude the original one -->
- <exclude>com.google.protobuf:protobuf-java</exclude>
- </excludes>
- </artifactSet>
<relocations>
<relocation>
<pattern>com.google.protobuf</pattern>
@@ -177,11 +186,4 @@
</plugin>
</plugins>
</build>
- <dependencies>
- <dependency>
- <groupId>com.google.protobuf</groupId>
- <artifactId>protobuf-java</artifactId>
- <version>${protobuf.version}</version>
- </dependency>
- </dependencies>
</project>