Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up BND instructions #1468

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Bananeweizen
Copy link
Contributor

  • Remove jcni version (it's been removed in 2017 from the code base).
  • Remove api version (it's never been used since moving the code to eclipse).
  • Set a fixed Bundle-Developer in generated manifests to avoid bnd-maven-plugin complaining about all the developers in the parent pom.xml not having a mandatory ID.
  • Set the Bundle-Description header in BND. Otherwise the pom.xml description is taken for that header, and unfortunately the pom.xml description tag contains whitespace from the XML formatting and indenting.
  • Use the versionmask macro instead of the version macro. version is an alias for versionmask that is for compatibility only and should not be used.

If you are wondering what the above means:
grafik
grafik

* Remove jcni version (it's been removed in 2017 from the code base).
* Remove api version (it's never been used since moving the code to
eclipse).
* Set a fixed Bundle-Developer in generated manifests to avoid
bnd-maven-plugin complaining about all the developers in the parent
pom.xml not having a mandatory ID.
* Set the Bundle-Description header in BND. Otherwise the pom.xml
description is taken for that header, and unfortunately the pom.xml
description tag contains whitespace from the XML formatting and
indenting.
* Use the versionmask macro instead of the version macro. version is an
alias for versionmask that is for compatibility only and should not be
used.
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.

None yet

1 participant