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

Fix native libs other than macos being copied correctly. #1624

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

cmelchior
Copy link
Contributor

I made a mistake in #1607 so we didn't copy native libs for Linux and Windows correctly. It only worked on CI because the MacOS JVM libs where in the correct place because of them being built in-place on Jenkins and we do not run Windows/Linux tests there.

But it was caught by our Github Actions branch.

I added an extra debug log that makes it easier to check for in the future.

@cmelchior cmelchior marked this pull request as ready for review January 11, 2024 12:34
@cmelchior cmelchior merged commit fd54a30 into main Jan 11, 2024
4 of 5 checks passed
@cmelchior cmelchior deleted the cm/fix-copying-native-libs branch January 11, 2024 13:01
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants