#2 add jackson component and its runtime deps (jsr-311 and stax2-api)
Closed 6 years ago by sgallagh. Opened 6 years ago by nphilipp.
modules/ nphilipp/maven f27--add-jackson-and-deps  into  f27

file modified
+15
@@ -556,6 +556,11 @@ 

                  ref: bdff74a

                  rationale: >

                      Build dependency of jdom2, msv.

+             # jackson-1.9.11-12.fc27

+             jackson:

+                 ref: 6691276

+                 rationale: >

+                     Runtime dependency of hadoop*, jersey*, resteasy.

              # jackson-annotations-2.7.6-4.fc28.1

              jackson-annotations:

                  ref: 4b328d0
@@ -859,6 +864,11 @@ 

                      Build dependency of guava, maven-assembly-plugin,

                           maven-enforcer, maven-filtering, maven-shared-utils,

                           maven-surefire, plexus-archiver, plexus-io, testng.

+             # jsr-311-1.1.1-14

+             jsr-311:

+                 ref: 378253c

+                 rationale: >

+                     Runtime dependency of jackson.

              # jtidy-1.0-0.25.20100930svn1125.fc28.1

              jtidy:

                  ref: 0dd587f
@@ -1781,6 +1791,11 @@ 

                  rationale: >

                      Build dependency of plexus-build-api, plexus-cipher,

                           plexus-sec-dispatcher.

+             # stax2-api-4.0.0-3

+             stax2-api:

+                 ref: 5cb4f86

+                 rationale: >

+                     Runtime dependency of jackson.

              # testng-6.12-2.fc28.1

              testng:

                  ref: 98433eb

Pull-Request has been closed by sgallagh

6 years ago

Why jackson was added here? If it's not needed by Maven or anything else in this module, then I don't see any reason for putting it there.

@mizdebsk I've been asking that myself, unfortunately @karsten is on vacation for a few days. I've been toying with the idea to move jackson out into its own module but for a quick fix will move it and its deps back into resteasy. Sorry for the noise.

I've removed jackson and all related packages from both master and f27.

Metadata