Skip to content

Commit

Permalink
Include java-xmlbuilder in tiered-storage-jcloud NAR (apache#3928) (a…
Browse files Browse the repository at this point in the history
…pache#3929)

In the POM file for tiered-storage-jcloud, the dependency on
com.jamesmurty.utils/java-xmlbuilder was marked with test scope, but in
fact this dependency is needed during normal use. This commit changes
the POM file so that the java-xmlbuilder dependency is packaged in the
NAR file.
  • Loading branch information
chairmank authored and merlimat committed Mar 29, 2019
1 parent ee76078 commit ec3d635
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions tiered-storage/jcloud/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,11 @@
<groupId>com.amazonaws</groupId>
<artifactId>aws-java-sdk-core</artifactId>
</dependency>
<dependency>
<groupId>com.jamesmurty.utils</groupId>
<artifactId>java-xmlbuilder</artifactId>
<version>1.1</version>
</dependency>
<dependency>
<groupId>org.apache.pulsar</groupId>
<artifactId>managed-ledger-original</artifactId>
Expand All @@ -84,12 +89,6 @@
<!-- we excluded the transitive dependencies above to make it work for standalone.
however it will cause test failures. bring following dependencies back for testing.
all these should be gone once we introduce NAR package for offloaders. -->
<dependency>
<groupId>com.jamesmurty.utils</groupId>
<artifactId>java-xmlbuilder</artifactId>
<version>1.1</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>net.iharder</groupId>
<artifactId>base64</artifactId>
Expand Down

0 comments on commit ec3d635

Please sign in to comment.