ServiceMix
  1. ServiceMix
  2. SM-2238

Remove overwritten activemq features from ServiceMix features.xml

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.0.0
    • Component/s: None
    • Labels:
      None

      Description

      The ServiceMix features.xml still contains some overwritten activemq features which were created as workaround for AMQ-4341, which has been solved in ActiveMQ 5.9.0. The features should be removed and those provided by ActiveMQ should be used

      1. 0003-SM-2238.patch
        8 kB
        Krzysztof Sobkowiak

        Activity

        Hide
        Krzysztof Sobkowiak added a comment - - edited

        I have removed the activemq, activemq-broker and activemq-client features from features.xml. I had one more problem after this change. ServiceMix hung after the first start. I had to kill it and start again. After the next start the I have mentioned another problem - log:display has displayed nothing.

        I have replaced the activemq-broker by activemq-broker-noweb in featuresBoot and it has solved my problem. I think, there is no reason to install activemq-web-console at boot. If one needs the console, it can be installed manually. From the same reason the patch replaces the dependency to activemq-broker by dependency to activemq-broker-noweb in other servicemix features.

        Show
        Krzysztof Sobkowiak added a comment - - edited I have removed the activemq, activemq-broker and activemq-client features from features.xml. I had one more problem after this change. ServiceMix hung after the first start. I had to kill it and start again. After the next start the I have mentioned another problem - log:display has displayed nothing. I have replaced the activemq-broker by activemq-broker-noweb in featuresBoot and it has solved my problem. I think, there is no reason to install activemq-web-console at boot. If one needs the console, it can be installed manually. From the same reason the patch replaces the dependency to activemq-broker by dependency to activemq-broker-noweb in other servicemix features.
        Hide
        Gert Vanthienen added a comment -

        Changes applied with thanks in http://svn.apache.org/viewvc?view=revision&revision=1568823

        I was unable to apply this patch directly to the trunk codebase, so I copied all the changes manually - could you just do a quick check to ensure I have not missed any bits?

        Thanks to Krzysztof for the patch!

        Show
        Gert Vanthienen added a comment - Changes applied with thanks in http://svn.apache.org/viewvc?view=revision&revision=1568823 I was unable to apply this patch directly to the trunk codebase, so I copied all the changes manually - could you just do a quick check to ensure I have not missed any bits? Thanks to Krzysztof for the patch!
        Hide
        Krzysztof Sobkowiak added a comment -

        You have performed all necessary changes. Thanks

        Show
        Krzysztof Sobkowiak added a comment - You have performed all necessary changes. Thanks

          People

          • Assignee:
            Gert Vanthienen
            Reporter:
            Krzysztof Sobkowiak
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development