Skip to content

Conversation

@paulrutter
Copy link
Contributor

  • Upgrade all dependencies, including bnd 7.1.0

I will first test this release myself before merging it, as the bnd 7.1.0 mentions a change that might be relevant / no longer backwards compatible. See https://github.com/bndtools/bnd/wiki/Changes-in-7.1.0#bnd

Improvement / Fix: Import-Package (coming from Export-Package) are not automatically added to generated Manifest anymore, if the export does not specify a version. This now conforms better to the [specification](https://docs.osgi.org/specification/osgi.core/8.0.0/framework.module.html#i3238802) which requires a version on exports. Notice that this results in different generated Manifests if exports do not specify a version. (PR #6270)

- Upgrade all dependencies, including bnd 7.1.0
@paulrutter paulrutter marked this pull request as draft December 1, 2024 17:24
- Reset maven version to 3.3.9, as specified in bnd 7.1.0 changelog
- Fix tests; add build timestamp greater than 1980 (required in new maven-archiver)
- Fix tests; change bnd version
- Don't lower maven version used earlier
- Add integration test for Export-Package / Import-Package, based on bndtools/bnd#6270
- This is backwards compatible, as maven-bundle-plugin automatically adds a version for all Export-Package statements if not specified (1.0.0)

These instructions:
```
<Export-Package>org.apache.felix.test1,org.apache.felix.test2;version="1-SNAPSHOT"</Export-Package>
<Import-Package>org.apache.felix.test1,org.apache.felix.test2</Import-Package>
```

Lead to these manifest entries:
```
Export-Package: org.apache.felix.test1;version="1.0.0",org.apache.felix.
 test2;version="1.0.0.SNAPSHOT"
Import-Package: org.apache.felix.test1,org.apache.felix.test2;version="[
 1.0,2)"
```
@paulrutter paulrutter marked this pull request as ready for review January 5, 2025 08:11
@paulrutter
Copy link
Contributor Author

I made some additional changes and added a integration test for bndtools/bnd#6270, to verify impact on the maven-bundle-plugin (there is no impact).

This PR is now ready for review.

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.

1 participant