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

chore(pkgbuild): delete existing app during preinstall, spctl --assess during postinstall #112

Merged
merged 3 commits into from
Mar 18, 2025

Conversation

ethanndickson
Copy link
Member

@ethanndickson ethanndickson commented Mar 13, 2025

Relates to #83.

It looks like deleting the app does indeed kill the NE process, so we should do that during preinstall.

In case the XPC issue we've been seeing is due to a race between the app being opened, and Gatekeeper ingesting the notarization ticket, we'll also force Gatekeeper to read the ticket by running spctl -a on the app bundle, and the extension bundle. The latter always fails, but an attempt to read it can't hurt.

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ethanndickson ethanndickson requested a review from ThomasK33 March 13, 2025 03:40
@ethanndickson ethanndickson marked this pull request as ready for review March 13, 2025 03:40
@ethanndickson ethanndickson self-assigned this Mar 13, 2025
Copy link
Member Author

ethanndickson commented Mar 18, 2025

Merge activity

  • Mar 17, 10:33 PM EDT: A user started a stack merge that includes this pull request via Graphite.
  • Mar 17, 10:33 PM EDT: A user merged this pull request with Graphite.

@ethanndickson ethanndickson merged commit 6947811 into main Mar 18, 2025
4 checks passed
@ethanndickson ethanndickson deleted the ethan/pkgbuild-clean-script branch March 18, 2025 02:33
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