Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345047 - ScanningAppProvider.setMonitoredDir(String) had an API change
Summary: ScanningAppProvider.setMonitoredDir(String) had an API change
Status: RESOLVED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: server (show other bugs)
Version: 7.4.0   Edit
Hardware: All All
: P3 critical (vote)
Target Milestone: 7.2.x   Edit
Assignee: Greg Wilkins CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-06 23:23 EDT by Hugues Malphettes CLA
Modified: 2011-05-09 21:35 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Hugues Malphettes CLA 2011-05-06 23:23:18 EDT 
When upgrading to 7.4.0 the older jetty.xml files won't work if they use an AppProvider that extends the ScanningAppProvider.
It appears that the method
setMonitoredDir(String)
   was renamed
setMonitoredDirName(String)

The workaround consists of changing the affected jetty.xml files.
Should we:
1- rollback the change?
2- put back the old method with an @deprecated?
3- let it like this and document?

For my case where the same config file can be used with different versions of jetty, it would be nice to choose #2.
Comment 1 Greg Wilkins CLA 2011-05-09 21:35:50 EDT 
r3102 for 7.4.1