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

Ship XQuartz libraries #5346

Merged
merged 2 commits into from
Dec 12, 2023
Merged

Conversation

boutinb
Copy link
Contributor

@boutinb boutinb commented Dec 8, 2023

Fixes https://github.com/jasp-stats/jasp-test-release/issues/2467

Load XQuartz package
Copy only the useful libraries
install_name_tool the libraries and the modules/R_X11.so and R_de.so files
Sign them

@boutinb boutinb requested a review from JorisGoosen December 8, 2023 17:07
@boutinb
Copy link
Contributor Author

boutinb commented Dec 11, 2023

I have tested it on Mac ARM & Intel.

@JorisGoosen
Copy link
Contributor

Ive turned on nightly builds so I can test it like that.

Load XQuartz package
Copy only the useful libraries
install_name_tool the libraries and the modules/R_X11.so and R_de.so files
Sign them
@boutinb
Copy link
Contributor Author

boutinb commented Dec 12, 2023

Ive turned on nightly builds so I can test it like that.

The modules need to be updated first (for jaspBase). I just did. I'm starting the nightly again.

@JorisGoosen
Copy link
Contributor

Why did you not mention the changes to jaspBase here then?

@boutinb
Copy link
Contributor Author

boutinb commented Dec 12, 2023

Why did you not mention the changes to jaspBase here then?

Yes I should have done. I thought jaspBase was already updated in the shipXQuartz branch

@JorisGoosen
Copy link
Contributor

Very nice!

@JorisGoosen JorisGoosen merged commit f7fbf2b into jasp-stats:development Dec 12, 2023
1 check passed
JorisGoosen pushed a commit that referenced this pull request Jan 2, 2024
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