Skip to content

Remove imports for java.* packages#896

Merged
kwin merged 1 commit into
developfrom
bugfix/remove-package-imports-for-java
Jun 30, 2026
Merged

Remove imports for java.* packages#896
kwin merged 1 commit into
developfrom
bugfix/remove-package-imports-for-java

Conversation

@kwin

@kwin kwin commented Jun 30, 2026

Copy link
Copy Markdown
Member

This is necessary to support AEM 6.5.x (shipping with Felix 6.0.2) where not all provided java packages are properly exported. This affects e.g. java.sql. Further info in
https://issues.apache.org/jira/browse/FELIX-5843

This closes #865

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the bundle’s BND configuration to avoid generating Import-Package entries for java.* packages, addressing Felix 6.0.2’s incomplete java.* exports on AEM 6.5.x and preventing runtime resolution failures (e.g., java.sql).

Changes:

  • Configure BND to suppress java.* package imports via -noimportjava: true.
  • Document the Felix/AEM motivation and link to FELIX-5843 for future maintainers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OleksandrBarybin OleksandrBarybin self-requested a review June 30, 2026 15:14
This is necessary to support AEM 6.5.x (shipping with Felix 6.0.2) where
not all provided java packages are properly exported. This affects e.g.
java.sql. Further info in
https://issues.apache.org/jira/browse/FELIX-5843

This closes #865
@kwin kwin force-pushed the bugfix/remove-package-imports-for-java branch from e96f24b to eff229b Compare June 30, 2026 16:33
@sonarqubecloud

Copy link
Copy Markdown

@kwin kwin merged commit cddb9a1 into develop Jun 30, 2026
14 checks passed
@kwin kwin deleted the bugfix/remove-package-imports-for-java branch June 30, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4.0 not working on AEM 6.5.22 due to unresolvable imports on java.sql

3 participants