GeoServer

Remove Mortbay package from geoserver main module

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7.4, 2.0-alpha2
  • Fix Version/s: 1.7.5, 2.0-beta1
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

Being mortbay no more used from geoserver main module can we delete it?

POST EDIT:
I'm referring to the main module, where is a "src\main\java\org\mortbay" source package.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Andrea Aime added a comment -

Mortbay... that would be Jetty related no?
But I don't understand what you mean by "package". Is this a real source package, you mean a jar dependency, what?

Show
Andrea Aime added a comment - Mortbay... that would be Jetty related no? But I don't understand what you mean by "package". Is this a real source package, you mean a jar dependency, what?
Hide
Permalink
Daniele Romagnoli added a comment - - edited

Yes, I mean the "mortbay" package (with org prefix) on "main\src\main\java\" source folder.

Show
Daniele Romagnoli added a comment - - edited Yes, I mean the "mortbay" package (with org prefix) on "main\src\main\java\" source folder.
Hide
Permalink
Andrea Aime added a comment -

Yeah, as far as I can see that package contains dead code, hasn't been used in years.
I'd say go ahead and remove it, but record the revision at which you removed it in this jira, so that it can be easily resurrected in case I'm missing a significant use of these classes.

Show
Andrea Aime added a comment - Yeah, as far as I can see that package contains dead code, hasn't been used in years. I'd say go ahead and remove it, but record the revision at which you removed it in this jira, so that it can be easily resurrected in case I'm missing a significant use of these classes.
Hide
Permalink
Daniele Romagnoli added a comment -

DONE.

revision 12325 on trunk.
revision 12326 on 1.7.x

Show
Daniele Romagnoli added a comment - DONE. revision 12325 on trunk. revision 12326 on 1.7.x
Hide
Permalink
Andrea Aime added a comment -

Mass closing all resolved issues that have been closed for more than one month without further comments

Show
Andrea Aime added a comment - Mass closing all resolved issues that have been closed for more than one month without further comments

People

  • Assignee:
    Daniele Romagnoli
    Reporter:
    Daniele Romagnoli
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: