This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | Don't scan $cluster/lib folder for other than platform clusters | ||
---|---|---|---|
Product: | apisupport | Reporter: | Jaroslav Tulach <jtulach> |
Component: | Harness | Assignee: | Jaroslav Tulach <jtulach> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | jglick |
Priority: | P2 | ||
Version: | 6.x | ||
Hardware: | Other | ||
OS: | Linux | ||
Issue Type: | DEFECT | Exception Reporter: |
Description
Jaroslav Tulach
2010-06-07 10:00:21 UTC
core-main#f002b057a516 Please consider backporting to closest post 6.9 release or some of its fix. This directly contradicts the intent of bug #185239. Perhaps MLP can be made to look up JAR paths from config/Modules/*.xml rather than assuming all {lib,core,modules}/*.jar are modules. I am not sure what other code might make this assumption. MPL does look for config/Modules/*.xml since 6.9. Unfortunately there is no config/Modules/*.xml for JARs in platform/lib, neither in $cluster/core/, so these dirs have to still be scanned. The first only for platform, the later, for each cluster. I have no opinion about bug 185239. Only that making trade-offs, just to make JavaFX run is not really beneficial for the future. Integrated into 'main-golden', will be available in build *201006080001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main/rev/f002b057a516 User: Jaroslav Tulach <jtulach@netbeans.org> Log: #187259: Skip lib directory in any other than platform cluster On second thought I guess it's not a problem. While lib/*.jar could be added to the classpath from other clusters (*), these JARs would not typically be modules anyway. (*) This would not be only for the benefit of JavaFX; it is occasionally necessary to put something in the app CP. (In reply to comment #5) > these JARs would not typically be modules anyway. True. Good reasoning. Transplanted to release691 as bcf74b96ce0f |