Maven 1.x Jetty Plugin

update to latest Jetty

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.2
  • Component/s: None
  • 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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

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