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

feat: Add jextract candidate #695

Merged
merged 1 commit into from
Aug 4, 2024
Merged

Conversation

helpermethod
Copy link
Member

@marc0der @aalmiray

The jextract guys finally moved the binary to bin and make use of an embedded JDK which resides in a different folder.

@github-actions github-actions bot requested a review from marc0der May 30, 2024 10:53
@helpermethod
Copy link
Member Author

Hi @marc0der! We still need to make some changes to the sdkman-hooks project, right? Or are they still in place?

@helpermethod helpermethod force-pushed the feat/add-jextract-candidate branch from 6d7f36c to ab0e24b Compare May 31, 2024 06:35
@helpermethod
Copy link
Member Author

Squashed all commits for a cleaner history.

@helpermethod
Copy link
Member Author

I also gave it a go on my machine and it actually creates a Java binding for a corresponding C header.

image

@helpermethod
Copy link
Member Author

@marc0der Can we merge this please? I need to get this out of my head 😆.

@marc0der
Copy link
Member

marc0der commented Aug 4, 2024

Sorry, just saw this now! Happy for you to merge it @helpermethod .

@helpermethod helpermethod merged commit 3a16ecb into master Aug 4, 2024
1 check passed
@marc0der marc0der deleted the feat/add-jextract-candidate branch August 4, 2024 20:52
@helpermethod
Copy link
Member Author

@marc0der For some reason the candidate doesn't show up. Any idea why?

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