Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.2.0
    • Fix Version/s: 4.3.0
    • Component/s: Bundles
    • Labels:
      None

      Description

      Update the servicemix bundle for jsmpp to 2.1.0.

      This version of jsmpp fixes a serious issue that affects the EnquireLink functionality of camel-smpp.

      The jsmpp release is available here: http://code.google.com/p/jsmpp/downloads/list

      1. SMX4-531.patch
        5 kB
        Christian Müller

        Issue Links

          Activity

          Hide
          Christian Müller added a comment -

          provide the patch.

          Because this dependency is currently not provided in a maven repository, if had first to install it manually:

          mvn install:install-file -DgroupId=com.google.code -DartifactId=jsmpp -Dversion=2.1.0 -Dpackaging=jar -Dfile=jsmpp-2.1.0.jar

          Show
          Christian Müller added a comment - provide the patch. Because this dependency is currently not provided in a maven repository, if had first to install it manually: mvn install:install-file -DgroupId=com.google.code -DartifactId=jsmpp -Dversion=2.1.0 -Dpackaging=jar -Dfile=jsmpp-2.1.0.jar
          Hide
          Gert Vanthienen added a comment -

          Fixed in http://svn.apache.org/viewvc?view=revision&revision=949957
          A snapshot has been deployed to repository.apache.org

          Show
          Gert Vanthienen added a comment - Fixed in http://svn.apache.org/viewvc?view=revision&revision=949957 A snapshot has been deployed to repository.apache.org
          Hide
          Willem Jiang added a comment -

          @Gert

          I just ran "mvn clean install -Pvalidate " in trunk/plateform/karaf directory, and found the feature validate complain about these

          [WARNING] No export found to match org.testng (imported by mvn:org.apache.servicemix.bundles/org.apache.servicemix.bundles.jsmpp/2.1.0_1-SNAPSHOT)
          [WARNING] No export found to match org.testng.annotations (imported by mvn:org.apache.servicemix.bundles/org.apache.servicemix.bundles.jsmpp/2.1.0_1-SNAPSHOT)
          

          I checked the jsmpp jar and found there are some test classes in org.jsmpp.bean, and they are using the testng class.
          So I marked the "org.testng" import package as optional in my patch, so please apply it into ServiceMix bundle repository.

          Here is the patch

          Index: pom.xml
          ===================================================================
          --- pom.xml	(revision 954397)
          +++ pom.xml	(working copy)
          @@ -44,6 +44,7 @@
                   </servicemix.osgi.export.pkg>
                   <servicemix.osgi.import.pkg>
                       !org.jsmpp*,
          +            org.testng.*;resolution:=optional,
                       *
                   </servicemix.osgi.import.pkg>
                   <servicemix.osgi.private.pkg>
          
          
          Show
          Willem Jiang added a comment - @Gert I just ran "mvn clean install -Pvalidate " in trunk/plateform/karaf directory, and found the feature validate complain about these [WARNING] No export found to match org.testng (imported by mvn:org.apache.servicemix.bundles/org.apache.servicemix.bundles.jsmpp/2.1.0_1-SNAPSHOT) [WARNING] No export found to match org.testng.annotations (imported by mvn:org.apache.servicemix.bundles/org.apache.servicemix.bundles.jsmpp/2.1.0_1-SNAPSHOT) I checked the jsmpp jar and found there are some test classes in org.jsmpp.bean, and they are using the testng class. So I marked the "org.testng" import package as optional in my patch, so please apply it into ServiceMix bundle repository. Here is the patch Index: pom.xml =================================================================== --- pom.xml (revision 954397) +++ pom.xml (working copy) @@ -44,6 +44,7 @@ </servicemix.osgi.export.pkg> <servicemix.osgi. import .pkg> !org.jsmpp*, + org.testng.*;resolution:=optional, * </servicemix.osgi. import .pkg> <servicemix.osgi. private .pkg>
          Hide
          Gert Vanthienen added a comment -

          Patch applied with thanks to Willem in http://svn.apache.org/viewvc?view=revision&revision=955163
          A new snapshot has been deployed as well by Hudson by now.

          Show
          Gert Vanthienen added a comment - Patch applied with thanks to Willem in http://svn.apache.org/viewvc?view=revision&revision=955163 A new snapshot has been deployed as well by Hudson by now.

            People

            • Assignee:
              Gert Vanthienen
              Reporter:
              Jaco van Tonder
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development