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

Update bouncycastle to jdk18on-1.76. #6517

Merged
merged 1 commit into from
Oct 3, 2023
Merged

Conversation

mbien
Copy link
Member

@mbien mbien commented Oct 3, 2023

  • switches from jdk15on to jdk18on (which is for JDK 8+, see page)
  • updates from 1.70 to 1.76

we might need this update for #6500, lets see if everything is green

 - switches from jdk15on to jdk18on (which is for JDK 8+)
 - updates from 1.70 to 1.76
@mbien mbien added Upgrade Library Library (Dependency) Upgrade ci:all-tests [ci] enable all tests labels Oct 3, 2023
@mbien mbien added this to the NB20 milestone Oct 3, 2023
@mbien mbien marked this pull request as ready for review October 3, 2023 07:15
Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Looks sane to me. Thank you.

@mbien mbien merged commit 2ef8fb0 into apache:master Oct 3, 2023
@mbien
Copy link
Member Author

mbien commented Oct 4, 2023

@matthiasblaesing the lib wrappers here don't expose any packages:

<configuration>
<data xmlns="http://www.netbeans.org/ns/nb-module-project/2">
<code-name-base>bcpg</code-name-base>
<module-dependencies/>
<public-packages/>
<class-path-extension>
<runtime-relative-path>bcpg.jar</runtime-relative-path>
<binary-origin>external/bcpg-jdk18on-1.76.jar</binary-origin>
</class-path-extension>
</data>
</configuration>

how does this work at runtime? Shouldn't this be <friend-packages> or shared via impl dependency?

@matthiasblaesing
Copy link
Contributor

matthiasblaesing commented Oct 4, 2023

The important part is here:

release.external/bcpg-jdk18on-1.76.jar=modules/bcpg.jar

This copies the bouncy castle PGP implementation into the modules folder of the corresponding cluster. In this case that is ide/modules. The important part is, that the library is an OSGI jar and that is picked up by the module system. So the "project" in the netbeans sources only exists as a place holder, the only interesting part is the copy operation. The build process itself is deactivated (the jar target is overwritten with an empty implementation):

<target name="jar"/>

At least that is my understanding.

@mbien
Copy link
Member Author

mbien commented Oct 5, 2023

thanks for the help - much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:all-tests [ci] enable all tests Upgrade Library Library (Dependency) Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants