Skip to content

fix: Application bundle integrity issues on macOS 13 #304

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 1 commit into from
Oct 31, 2022

Conversation

mayeut
Copy link
Contributor

@mayeut mayeut commented Oct 30, 2022

This commit unbundles the macOS app to keep only what's really needed & get rid of the GUI stuff.

fix #301

@henryiii
Copy link
Contributor

That seem reasonable. I know that the zip module in Python does not support symbolic links. Scikit-build doesn’t handle them either. This would explain the “damaged” message.

This commit unbundles the macOS app to keep only what's really needed & get rid of the GUI stuff.
@mayeut mayeut force-pushed the macos-unbundle-app branch from 3693ec2 to 71330d8 Compare October 30, 2022 17:40
@mayeut mayeut marked this pull request as ready for review October 30, 2022 17:40
@mayeut
Copy link
Contributor Author

mayeut commented Oct 30, 2022

I'll try to use it a bit more but some first tests on my laptop with the wheel produced by GHA works.

Copy link
Contributor

@henryiii henryiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixes the issue for me, too, using the wheel locally.

@henryiii henryiii merged commit 24919f2 into scikit-build:master Oct 31, 2022
@thewtex
Copy link
Member

thewtex commented Oct 31, 2022

Thank you @mayeut !

@mayeut mayeut deleted the macos-unbundle-app branch November 1, 2022 08:02
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.

Unsigned binary issues on macOS 13
3 participants