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: | Netigso.fakeBundle skips modules without public packages | ||
|---|---|---|---|
| Product: | platform | Reporter: | Jesse Glick <jglick> |
| Component: | Netigso | Assignee: | Jaroslav Tulach <jtulach> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | normal | ||
| Priority: | P4 | ||
| Version: | 7.0 | ||
| Hardware: | PC | ||
| OS: | Linux | ||
| Issue Type: | DEFECT | Exception Reporter: | |
| Bug Depends on: | |||
| Bug Blocks: | 198248 | ||
The reasoning is like this: There is no reason why an OSGi should want to depend on a module without public packages (OSGi does not support implementation dependencies). Thus there is no reason to export modules without public packages.
I verify the problem with man.getMainAttributes().putValue("Export-Package",
m.getCodeNameBase().
(In reply to comment #1) > There is no reason why an OSGi [bundle?] should want to > depend on a module without public packages Right, it cannot. But the app might expect that such a module is nonetheless loaded by the OSGi framework and activated, which would make an observable difference in case there is some system such as the Eclipse plugin.xml scanner which pays attention to the list of activated bundles (and changes therein). We will fix the problem as soon as it really demonstrates itself. |
if ("-".equals(exp)) { return null; } Does this mean that modules with no public packages are simply not loaded by the OSGi container at all, perhaps as a performance optimization? If true, then modules providing just services (e.g. an Eclipse plugin.xml) will be invisible to the OSGi framework, unless they are packaged as bundles rather than NB modules. Also the following: man.getMainAttributes().putValue("Export-Package", m.getCodeNameBase()); looks like nonsense. It is deprecated to not specify OpenIDE-Module-Public-Packages at all, so we do not really need to support that. The code should probably read: if (exp != null && !exp.equals("-")) { ...set Export-Package as now... }