Camel
  1. Camel
  2. CAMEL-4671

The camel-core feature should not reference jaxb2 api / impl

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.8.2
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      Description

      Those are supposed to be provided by the JRE and deploying those cause problems when the osgi does not hide those packages.

        Activity

        Guillaume Nodet created issue -
        Guillaume Nodet made changes -
        Field Original Value New Value
        Assignee Guillaume Nodet [ gnt ]
        Hide
        Guillaume Nodet added a comment -

        There are two cases here.
        When deploying in Karaf using a default JRE 1.6 profile, jaxb2 spec and impl are provided by the JRE and thus not required.
        When deploying on a JRE 1.5 profile (even using a 1.6 JRE, i.e. hiding the new packages), those will have to be deployed by the user.
        A possible alternative would be to create two different features, but I don't really think it's necessary.

        Show
        Guillaume Nodet added a comment - There are two cases here. When deploying in Karaf using a default JRE 1.6 profile, jaxb2 spec and impl are provided by the JRE and thus not required. When deploying on a JRE 1.5 profile (even using a 1.6 JRE, i.e. hiding the new packages), those will have to be deployed by the user. A possible alternative would be to create two different features, but I don't really think it's necessary.
        Guillaume Nodet <gnodet@apache.org> committed 5609836ce8bad7ab730c256db6d1c4f87285ee41 (1 file)
        Reviews: none

        [CAMEL-4671] The camel-core feature should not reference jaxb2 api / impl
        Conflicts:

        platforms/karaf/features/src/main/resources/features.xml

        git-svn-id: https://svn.apache.org/repos/asf/camel/branches/camel-2.8.x@1201766 13f79535-47bb-0310-9956-ffa450edef68

        gnodet committed 1201766 (1 file)
        Reviews: none

        [CAMEL-4671] The camel-core feature should not reference jaxb2 api / impl

        Conflicts:

        platforms/karaf/features/src/main/resources/features.xml

        Guillaume Nodet <gnodet@apache.org> committed f4344c564c42293e721e6dc4eab008408eaa0fd0 (1 file)
        Reviews: none

        [CAMEL-4671] The camel-core feature should not reference jaxb2 api / impl
        git-svn-id: https://svn.apache.org/repos/asf/camel/trunk@1201768 13f79535-47bb-0310-9956-ffa450edef68

        gnodet committed 1201768 (1 file)
        Reviews: none

        [CAMEL-4671] The camel-core feature should not reference jaxb2 api / impl

        Guillaume Nodet made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.9.0 [ 12316374 ]
        Resolution Fixed [ 1 ]
        Claus Ibsen <davsclaus@apache.org> committed c693a40472644a56d513152b1b13a62c85c04752 (1 file)
        Reviews: none

        CAMEL-4671: Fixed typo in features.xml
        git-svn-id: https://svn.apache.org/repos/asf/camel/trunk@1201811 13f79535-47bb-0310-9956-ffa450edef68

        Claus Ibsen committed 1201811 (1 file)
        J. Daniel Kulp <dkulp@apache.org> committed 1ecf6cf6eb1eaff88494ef2e90c7e2475868b564 (0 files)
        Reviews: none

        Merged revisions 1201811 via svnmerge from https://svn.apache.org/repos/asf/camel/trunk
        ........
          r1201811 | davsclaus | 2011-11-14 12:47:49 -0500 (Mon, 14 Nov 2011) | 1 line
          
          CAMEL-4671: Fixed typo in features.xml
        ........


        git-svn-id: https://svn.apache.org/repos/asf/camel/branches/camel-2.8.x@1201885 13f79535-47bb-0310-9956-ffa450edef68

        Daniel Kulp committed 1201885 (1 file)
        Reviews: none

        Merged revisions 1201811 via svnmerge from
        https://svn.apache.org/repos/asf/camel/trunk

        ........
          r1201811 | davsclaus | 2011-11-14 12:47:49 -0500 (Mon, 14 Nov 2011) | 1 line
          
          CAMEL-4671: Fixed typo in features.xml
        ........

        J. Daniel Kulp <dkulp@apache.org> committed 0f016b1917d0633a807c650b18ebc3c5fa406c75 (0 files)
        Reviews: none

        Recorded merge of revisions 1198520,1198526,1200144,1200869,1201767-1201768 via svnmerge from https://svn.apache.org/repos/asf/camel/trunk
        ........
          r1198520 | cmueller | 2011-11-06 12:55:34 -0500 (Sun, 06 Nov 2011) | 1 line
          
          CAMEL-4632: Upgrade org.apache.servicemix.bundles:org.apache.servicemix.bundles.snmp4j 1.8.1_3 -> 1.8.1_5
        ........
          r1198526 | cmueller | 2011-11-06 13:44:23 -0500 (Sun, 06 Nov 2011) | 1 line
          
          CAMEL-4633: Upgrade net.sf.dozer:dozer 5.3.1 -> 5.3.2
        ........
          r1200144 | cmueller | 2011-11-09 23:30:30 -0500 (Wed, 09 Nov 2011) | 1 line
          
          CAMEL-4654: Upgrade the org.apache.servicemix.bundles to the lates hotfix versions
        ........
          r1200869 | davsclaus | 2011-11-11 08:14:27 -0500 (Fri, 11 Nov 2011) | 1 line
          
          CAMEL-4668: Fixed JPA consumer to rollback if one Exchange failed
        ........
          r1201767 | gnodet | 2011-11-14 11:01:03 -0500 (Mon, 14 Nov 2011) | 1 line
          
          [CAMEL-4670] The javax.xml.transform.stax imported package shoule be optional
        ........
          r1201768 | gnodet | 2011-11-14 11:01:24 -0500 (Mon, 14 Nov 2011) | 1 line
          
          [CAMEL-4671] The camel-core feature should not reference jaxb2 api / impl
        ........


        git-svn-id: https://svn.apache.org/repos/asf/camel/branches/camel-2.8.x@1201887 13f79535-47bb-0310-9956-ffa450edef68

        Daniel Kulp committed 1201887 (1 file)
        Reviews: none

        Recorded merge of revisions 1198520,1198526,1200144,1200869,1201767-1201768 via svnmerge from
        https://svn.apache.org/repos/asf/camel/trunk

        ........
          r1198520 | cmueller | 2011-11-06 12:55:34 -0500 (Sun, 06 Nov 2011) | 1 line
          
          CAMEL-4632: Upgrade org.apache.servicemix.bundles:org.apache.servicemix.bundles.snmp4j 1.8.1_3 -> 1.8.1_5
        ........
          r1198526 | cmueller | 2011-11-06 13:44:23 -0500 (Sun, 06 Nov 2011) | 1 line
          
          CAMEL-4633: Upgrade net.sf.dozer:dozer 5.3.1 -> 5.3.2
        ........
          r1200144 | cmueller | 2011-11-09 23:30:30 -0500 (Wed, 09 Nov 2011) | 1 line
          
          CAMEL-4654: Upgrade the org.apache.servicemix.bundles to the lates hotfix versions
        ........
          r1200869 | davsclaus | 2011-11-11 08:14:27 -0500 (Fri, 11 Nov 2011) | 1 line
          
          CAMEL-4668: Fixed JPA consumer to rollback if one Exchange failed
        ........
          r1201767 | gnodet | 2011-11-14 11:01:03 -0500 (Mon, 14 Nov 2011) | 1 line
          
          [CAMEL-4670] The javax.xml.transform.stax imported package shoule be optional
        ........
          r1201768 | gnodet | 2011-11-14 11:01:24 -0500 (Mon, 14 Nov 2011) | 1 line
          
          [CAMEL-4671] The camel-core feature should not reference jaxb2 api / impl
        ........

        Willem Ning Jiang <ningjiang@apache.org> committed 6c08c5dfa238fb10b263c21742062beb86d3cd89 (3 files)
        Reviews: none

        Merged revisions 1202205-1202206,1202223 via svnmerge from https://svn.apache.org/repos/asf/camel/trunk
        ........
          r1202205 | ningjiang | 2011-11-15 22:22:30 +0800 (Tue, 15 Nov 2011) | 1 line
          
          CAMEL-4681 Add xml-spec-api feature into apache-camel
        ........
          r1202206 | ningjiang | 2011-11-15 22:23:32 +0800 (Tue, 15 Nov 2011) | 1 line
          
          CAMEL-4671 Fixed the osgi itest errors
        ........
          r1202223 | ningjiang | 2011-11-15 23:00:03 +0800 (Tue, 15 Nov 2011) | 1 line
          
          CAMEL-4681 set the xml-spec-api feature start level to be 10 as Dan suggested
        ........


        git-svn-id: https://svn.apache.org/repos/asf/camel/branches/camel-2.8.x@1202232 13f79535-47bb-0310-9956-ffa450edef68

        ningjiang committed 1202232 (4 files)
        Reviews: none

        Merged revisions 1202205-1202206,1202223 via svnmerge from
        https://svn.apache.org/repos/asf/camel/trunk

        ........
          r1202205 | ningjiang | 2011-11-15 22:22:30 +0800 (Tue, 15 Nov 2011) | 1 line
          
          CAMEL-4681 Add xml-spec-api feature into apache-camel
        ........
          r1202206 | ningjiang | 2011-11-15 22:23:32 +0800 (Tue, 15 Nov 2011) | 1 line
          
          CAMEL-4671 Fixed the osgi itest errors
        ........
          r1202223 | ningjiang | 2011-11-15 23:00:03 +0800 (Tue, 15 Nov 2011) | 1 line
          
          CAMEL-4681 set the xml-spec-api feature start level to be 10 as Dan suggested
        ........

        Hide
        Willem Jiang added a comment -

        This patch is revert in Camel 2.8.4 to maintain the backward compatibility of Camel 2.8.x
        http://svn.apache.org/viewvc?rev=1222332&view=rev
        http://svn.apache.org/viewvc?rev=1222435&view=rev

        Show
        Willem Jiang added a comment - This patch is revert in Camel 2.8.4 to maintain the backward compatibility of Camel 2.8.x http://svn.apache.org/viewvc?rev=1222332&view=rev http://svn.apache.org/viewvc?rev=1222435&view=rev
        Hide
        Jean-Baptiste Onofré added a comment -

        Even using a JRE 1.6 profiles (with Karaf 2.2.4), the JAXB bundles are required (version mismatch).

        Show
        Jean-Baptiste Onofré added a comment - Even using a JRE 1.6 profiles (with Karaf 2.2.4), the JAXB bundles are required (version mismatch).
        J. Daniel Kulp <dkulp@apache.org> committed 4941e79487a8c960a356a66b9860058a9f637efd (2 files)
        Reviews: none

        [CAMEL-4671] Reverting 1201768 (with permission from gnodet, the orinal author) due to issues it is causing as discussed on dev list.
        git-svn-id: https://svn.apache.org/repos/asf/camel/trunk@1224868 13f79535-47bb-0310-9956-ffa450edef68

        Daniel Kulp committed 1224868 (2 files)
        Reviews: none

        [CAMEL-4671] Reverting 1201768 (with permission from gnodet, the orinal
        author) due to issues it is causing as discussed on dev list.

        Daniel Kulp made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Daniel Kulp added a comment -


        This change is causing issues with several components. We're looking for better solutions that can be provided by future versions of Karaf.

        Show
        Daniel Kulp added a comment - This change is causing issues with several components. We're looking for better solutions that can be provided by future versions of Karaf.
        Daniel Kulp made changes -
        Fix Version/s 2.9.0 [ 12316374 ]
        Fix Version/s 2.8.3 [ 12318649 ]
        J. Daniel Kulp <dkulp@apache.org> committed 553a4fbdc21800641c366c9fff5aaef8dfeceb5d (0 files)
        Reviews: none

        Recorded merge of revisions 1222709,1222765,1222821,1222919,1222958,1224868 via svnmerge from https://svn.apache.org/repos/asf/camel/branches/camel-2.9.x
        ........
          r1222709 | jbonofre | 2011-12-23 10:56:36 -0500 (Fri, 23 Dec 2011) | 1 line
          
          [CAMEL-4818] camel-hdfs and camel-soap reference xml-specs-api feature
        ........
          r1222765 | jbonofre | 2011-12-23 12:35:22 -0500 (Fri, 23 Dec 2011) | 1 line
          
          [CAMEL-4815] camel-atom feature references the xml-specs-api feature
        ........
          r1222821 | jbonofre | 2011-12-23 15:00:51 -0500 (Fri, 23 Dec 2011) | 1 line
          
          [CAMEL-4819] camel-jibx and camel-xstream reference xml-specs-api feature
        ........
          r1222919 | jbonofre | 2011-12-24 01:32:42 -0500 (Sat, 24 Dec 2011) | 1 line
          
          [CAMEL-4828] Add woodstox dependencies in the xml-specs-api feature
        ........
          r1222958 | cmueller | 2011-12-24 06:07:46 -0500 (Sat, 24 Dec 2011) | 1 line
          
          removed the version attribute from the dependency to the Karaf provided eventadmin feature to be compliant with the other dependencies
        ........
          r1224868 | dkulp | 2011-12-26 22:22:57 -0500 (Mon, 26 Dec 2011) | 2 lines
          
          [CAMEL-4671] Reverting 1201768 (with permission from gnodet, the orinal
          author) due to issues it is causing as discussed on dev list.
        ........


        git-svn-id: https://svn.apache.org/repos/asf/camel/branches/camel-2.8.x@1227268 13f79535-47bb-0310-9956-ffa450edef68

        Daniel Kulp committed 1227268 (1 file)
        Reviews: none

        Recorded merge of revisions 1222709,1222765,1222821,1222919,1222958,1224868 via svnmerge from
        https://svn.apache.org/repos/asf/camel/branches/camel-2.9.x

        ........
          r1222709 | jbonofre | 2011-12-23 10:56:36 -0500 (Fri, 23 Dec 2011) | 1 line
          
          [CAMEL-4818] camel-hdfs and camel-soap reference xml-specs-api feature
        ........
          r1222765 | jbonofre | 2011-12-23 12:35:22 -0500 (Fri, 23 Dec 2011) | 1 line
          
          [CAMEL-4815] camel-atom feature references the xml-specs-api feature
        ........
          r1222821 | jbonofre | 2011-12-23 15:00:51 -0500 (Fri, 23 Dec 2011) | 1 line
          
          [CAMEL-4819] camel-jibx and camel-xstream reference xml-specs-api feature
        ........
          r1222919 | jbonofre | 2011-12-24 01:32:42 -0500 (Sat, 24 Dec 2011) | 1 line
          
          [CAMEL-4828] Add woodstox dependencies in the xml-specs-api feature
        ........
          r1222958 | cmueller | 2011-12-24 06:07:46 -0500 (Sat, 24 Dec 2011) | 1 line
          
          removed the version attribute from the dependency to the Karaf provided eventadmin feature to be compliant with the other dependencies
        ........
          r1224868 | dkulp | 2011-12-26 22:22:57 -0500 (Mon, 26 Dec 2011) | 2 lines
          
          [CAMEL-4671] Reverting 1201768 (with permission from gnodet, the orinal
          author) due to issues it is causing as discussed on dev list.
        ........

        Hide
        Claus Ibsen added a comment -

        Yeah for now lets avoid this change in the 2.x timeline, as it causes problems.

        Show
        Claus Ibsen added a comment - Yeah for now lets avoid this change in the 2.x timeline, as it causes problems.
        Claus Ibsen made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]

          People

          • Assignee:
            Guillaume Nodet
            Reporter:
            Guillaume Nodet
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development