Skip to content

Commit d241432

Browse files
committed
Properly manage the Woodstox dependency
Remove Woodstox as a direct dependency of webapp and distribution. Woodstox is a transitive dependency of Axiom, so we don't need to declare it as a direct dependency there. On the other hand, we should have a dependencyManagement entry for it because it's also a transitive dependency of other dependencies, in particular jaxws-rt. The dependencyManagement entry ensures we use the same version everywhere.
1 parent 075201b commit d241432

File tree

3 files changed

+5
-10
lines changed

3 files changed

+5
-10
lines changed

modules/distribution/pom.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -302,11 +302,6 @@
302302
<groupId>org.eclipse.angus</groupId>
303303
<artifactId>angus-mail</artifactId>
304304
</dependency>
305-
<dependency>
306-
<groupId>com.fasterxml.woodstox</groupId>
307-
<artifactId>woodstox-core</artifactId>
308-
<version>7.0.0</version>
309-
</dependency>
310305
<dependency>
311306
<groupId>org.apache.commons</groupId>
312307
<artifactId>commons-fileupload2-jakarta-servlet6</artifactId>

modules/webapp/pom.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -246,11 +246,6 @@
246246
<groupId>org.eclipse.angus</groupId>
247247
<artifactId>angus-activation</artifactId>
248248
</dependency>
249-
<dependency>
250-
<groupId>com.fasterxml.woodstox</groupId>
251-
<artifactId>woodstox-core</artifactId>
252-
<version>7.0.0</version>
253-
</dependency>
254249
<dependency>
255250
<groupId>org.apache.commons</groupId>
256251
<artifactId>commons-fileupload2-core</artifactId>

pom.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1011,6 +1011,11 @@
10111011
<artifactId>jacorb-omgapi</artifactId>
10121012
<version>3.9</version>
10131013
</dependency>
1014+
<dependency>
1015+
<groupId>com.fasterxml.woodstox</groupId>
1016+
<artifactId>woodstox-core</artifactId>
1017+
<version>7.0.0</version>
1018+
</dependency>
10141019

10151020
<!-- Jetty is used by some of the unit tests -->
10161021
<dependency>

0 commit comments

Comments
 (0)