Qpid
  1. Qpid
  2. QPID-4456

update the Java broker-only release package to account for changes in plugin system

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.19
    • Fix Version/s: 0.20
    • Component/s: Packaging
    • Labels:
      None

      Description

      Following the simplification of the plugin system in the Java broker via QPID-4335 and the modifications to the tests in QPID-4353, the plugins are not being packaged in the broker-only release binary (as per discussion on QPID-4444 just prior to the 0.20 beta).

      The build system should be updated to account for the changes and ensure the plugins are again included.

        Activity

        Robbie Gemmell created issue -
        Robbie Gemmell made changes -
        Field Original Value New Value
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Robbie Gemmell added a comment -
        Show
        Robbie Gemmell added a comment - Changes made in: http://svn.apache.org/viewvc?rev=1412359&view=rev
        Robbie Gemmell made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Hide
        Robbie Gemmell added a comment -

        Rob, could you review this change please?

        Thanks,
        Robbie

        Show
        Robbie Gemmell added a comment - Rob, could you review this change please? Thanks, Robbie
        Robbie Gemmell made changes -
        Assignee Robbie Gemmell [ gemmellr ] Rob Godfrey [ rgodfrey ]
        Hide
        Rob Godfrey added a comment -

        Looks good to me

        Show
        Rob Godfrey added a comment - Looks good to me
        Rob Godfrey made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Robbie Gemmell added a comment -

        Reopening because I forgot to update the qpid-server.bat file for Windows to also pick up the plugins.

        Show
        Robbie Gemmell added a comment - Reopening because I forgot to update the qpid-server.bat file for Windows to also pick up the plugins.
        Robbie Gemmell made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Rob Godfrey [ rgodfrey ] Robbie Gemmell [ gemmellr ]
        Hide
        Robbie Gemmell added a comment -

        Updated the windows script and the broker startup example in the documentation:
        http://svn.apache.org/viewvc?rev=1413444&view=rev

        Show
        Robbie Gemmell added a comment - Updated the windows script and the broker startup example in the documentation: http://svn.apache.org/viewvc?rev=1413444&view=rev
        Robbie Gemmell made changes -
        Status Reopened [ 4 ] In Progress [ 3 ]
        Robbie Gemmell made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Hide
        Robbie Gemmell added a comment -

        Hi Keith, could you review this change please?

        Show
        Robbie Gemmell added a comment - Hi Keith, could you review this change please?
        Robbie Gemmell made changes -
        Assignee Robbie Gemmell [ gemmellr ] Keith Wall [ k-wall ]
        Hide
        Justin Ross added a comment -

        The first change, from Robbie's initial comment, is reviewed by Rob and approved for 0.20.

        Show
        Justin Ross added a comment - The first change, from Robbie's initial comment, is reviewed by Rob and approved for 0.20.
        Hide
        Keith Wall added a comment -

        Changes look fine to me.

        Show
        Keith Wall added a comment - Changes look fine to me.
        Keith Wall made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Keith Wall made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Robbie Gemmell added a comment - - edited

        First change now merged to the 0.20 branch, second change requested for inclusion following review by Keith.

        Show
        Robbie Gemmell added a comment - - edited First change now merged to the 0.20 branch, second change requested for inclusion following review by Keith.
        Hide
        Justin Ross added a comment -

        Second change reviewed by Keith; approved for 0.20.

        Show
        Justin Ross added a comment - Second change reviewed by Keith; approved for 0.20.
        Hide
        Robbie Gemmell added a comment -

        Second commit now merged to the 0.20 release branch.

        Show
        Robbie Gemmell added a comment - Second commit now merged to the 0.20 release branch.
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        1h 32m 1 Robbie Gemmell 21/Nov/12 23:10
        Resolved Resolved Reopened Reopened
        3d 4h 26m 1 Robbie Gemmell 25/Nov/12 23:31
        Reopened Reopened In Progress In Progress
        2m 40s 1 Robbie Gemmell 25/Nov/12 23:34
        In Progress In Progress Reviewable Reviewable
        19h 46m 2 Robbie Gemmell 25/Nov/12 23:34
        Reviewable Reviewable Resolved Resolved
        23h 59m 2 Keith Wall 26/Nov/12 23:26
        Resolved Resolved Closed Closed
        15s 1 Keith Wall 26/Nov/12 23:26

          People

          • Assignee:
            Keith Wall
            Reporter:
            Robbie Gemmell
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development