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

Remove the quarantine bit from the serial command on macOS #159

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

kstrauser
Copy link
Contributor

The serial-macos-latest/serial command in the installer bundle has the quarantine bit set. This unsets that attribute so that macOS doesn't throw an error message like

“serial” Not Opened: Apple could not verify “serial” is free of malware that may harm your Mac or compromise your privacy.

This is the same error message @jullrich saw in #149. This fixes it. After the change, the installer script runs to completion on my M2 MacBook Air running macOS 15.3.2.

Screenshot of a macOS error message that says: "serial" Not Opened: Apple could not verify "serial" is free of malware that may harm your Mac or compromise your privacy.

@jullrich
Copy link

looks good. And yes, that is exactly the issue I had/fix I used.

@cooperq
Copy link
Collaborator

cooperq commented Mar 18, 2025

This looks fine to me in theory. I will test it ASAP.

@cooperq cooperq self-requested a review March 19, 2025 16:34
@cooperq cooperq self-assigned this Mar 19, 2025
@cooperq
Copy link
Collaborator

cooperq commented Mar 19, 2025

this looks great! Unfortunately there is a merge conflict here because i just merged another change that moved the install-linux and install-mac files into a unified install script. If you can update your changes for the latest HEAD of main I would appreciate it and be happy to accept this PR.

@kstrauser
Copy link
Contributor Author

Hey @cooperq, I've rebased it off the new combined script now.

@wgreenberg wgreenberg merged commit 88f81d8 into EFForg:main Mar 20, 2025
1 check passed
@wgreenberg
Copy link
Collaborator

thanks @kstrauser!

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.

4 participants