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

improvement: provide extra package for jre-native and jdk, jdk-native #57

Open
wants to merge 2 commits into
base: scarthgap
Choose a base branch
from

Conversation

GeVa2072
Copy link
Contributor

providing extra package on the bases of pre build version for jdk-21 and jdk-21-native

@GeVa2072
Copy link
Contributor Author

GeVa2072 commented Dec 3, 2024

Hi @sebveit,

Does this PR is ok for you, or do you have some remark ?
If it's ok, i will make other PR for other version of Java.

@sebveit
Copy link
Contributor

sebveit commented Dec 3, 2024

I'm going to look into this PR on friday. Sry for the delay!

@sebveit
Copy link
Contributor

sebveit commented Dec 6, 2024

I don't know why GitHub Workflow isn't working on your pull requests. Maybe you could check on your fork under settings, if you need to authorize it somehow?

Nevertheless, your PR looks nice! Only things I have spotted is that the contents of recipes-core/openjdk-21/openjdk-21-jdk.inc could be placed in recipes-core/openjdk-21/openjdk-21-jdk_21.0.4+7.bb, like you did with the JRE recipe.

I'm going to test the native JDK recipe of this PR with a recipe, that compiles a simple hello world, next week. If that works, I'm merging your changes.

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.

2 participants