RefuX commented on code in PR #75:
URL: https://github.com/apache/felix-atomos/pull/75#discussion_r1364877714
##########
atomos/src/main/java/org/apache/felix/atomos/impl/modules/AtomosModules.java:
##########
@@ -527,14 +527,28 @@ private void generateHeaders(Map<String, String> headers,
Module m)
// only do exports for non bundle modules
// real OSGi bundles already have good export capabilities
StringBuilder exportPackageHeader = new StringBuilder();
- desc.exports().stream().sorted().forEach((exports) -> {
- if (exportPackageHeader.length() > 0)
- {
- exportPackageHeader.append(", ");
- }
- exportPackageHeader.append(exports.source());
- // TODO map targets to x-friends directive?
- });
+
+ // ModuleDescriptor.exports() is empty for an automatic
module, which is different from
+ // JPMS at runtime where all packages in the automatic module
are exported
+ if (desc.isAutomatic()) {
+ desc.packages().stream().sorted().forEach((packages) -> {
+ if (exportPackageHeader.length() > 0)
+ {
+ exportPackageHeader.append(", ");
+ }
+ exportPackageHeader.append(packages);
+ });
+ }
+ else {
+ desc.exports().stream().sorted().forEach((exports) -> {
Review Comment:
Haha, I did find your naming convention _quite_ odd.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@felix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
