Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.2
    • Labels:
      None
    • Environment:
      all
    • Number of attachments :
      1

      Description

      This updates the plugin to use the latest Jetty and to retrieve Jetty jars from the default ibiblio Maven site.

        Activity

        Hide
        Adam Fisk added a comment -

        Here's the little patch to use the latest Jetty posted at ibiblio. They don't have 4.2.21, but this is good enough.

        Thanks Vincent.

        -Adam

        Show
        Adam Fisk added a comment - Here's the little patch to use the latest Jetty posted at ibiblio. They don't have 4.2.21, but this is good enough. Thanks Vincent. -Adam
        Hide
        Vincent Massol added a comment -

        Applied. Are you sure you don't need to upgrade the jasper versions? I hope your have tested your changed? Next time, could you please also attach the patch for changes.xml? Thanks.

        Show
        Vincent Massol added a comment - Applied. Are you sure you don't need to upgrade the jasper versions? I hope your have tested your changed? Next time, could you please also attach the patch for changes.xml? Thanks.
        Hide
        Adam Fisk added a comment -

        It's working fine for me with the older versions of jasper, but the Jetty folks include the 4.1.30 Jasper versions with their downloads, so it's probably a good idea to update to those. Here's the patch for doing so. I hope the changes.xml is the correct syntax. In particular, I kept the same issue ID since its a continuation of the same issue – hope that's alright.

        Thanks.

        -Adam

        Show
        Adam Fisk added a comment - It's working fine for me with the older versions of jasper, but the Jetty folks include the 4.1.30 Jasper versions with their downloads, so it's probably a good idea to update to those. Here's the patch for doing so. I hope the changes.xml is the correct syntax. In particular, I kept the same issue ID since its a continuation of the same issue – hope that's alright. Thanks. -Adam

          People

          • Assignee:
            Vincent Massol
            Reporter:
            Adam Fisk
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 5 minutes
              5m
              Remaining:
              Remaining Estimate - 5 minutes
              5m
              Logged:
              Time Spent - Not Specified
              Not Specified