Skip to content

fix mock release test #322

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

Merged
merged 31 commits into from
Nov 29, 2023
Merged

Conversation

DimitrisStaratzis
Copy link
Contributor

@DimitrisStaratzis DimitrisStaratzis commented Nov 22, 2023

The issue was that the step that was testing the release was using an outdated version of set-up-java and that was causing it to download an x86_64 version of Java in arm runners. I will open a new PR to add dependabot in this repo.

On top of that, the NativeLibLoader was lacking a / in the path to be able to locate the native libs in apple silicon macs.

@DimitrisStaratzis DimitrisStaratzis marked this pull request as draft November 22, 2023 10:40
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from 0c1736a to b44eaf7 Compare November 22, 2023 10:43
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from 5d03e9b to aae0679 Compare November 22, 2023 12:19
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from aae0679 to f59874f Compare November 22, 2023 12:28
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from 7c6d4c4 to 6cc1933 Compare November 27, 2023 11:03
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from bfc899a to e7dde5d Compare November 27, 2023 17:29
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from e7dde5d to 85896c4 Compare November 27, 2023 17:31
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from c1e422a to a190d65 Compare November 28, 2023 10:14
@DimitrisStaratzis DimitrisStaratzis force-pushed the dstara/fix_mock_release_testing branch from 4519877 to d3ef682 Compare November 28, 2023 11:08
@DimitrisStaratzis DimitrisStaratzis marked this pull request as ready for review November 28, 2023 11:30
@DimitrisStaratzis DimitrisStaratzis added the bug Something isn't working label Nov 28, 2023
@DimitrisStaratzis DimitrisStaratzis merged commit cefd812 into master Nov 29, 2023
@DimitrisStaratzis DimitrisStaratzis deleted the dstara/fix_mock_release_testing branch November 29, 2023 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants