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

Improve macOS installation documentation #15438

Merged
merged 1 commit into from Nov 10, 2019

Conversation

ilian
Copy link
Contributor

@ilian ilian commented Nov 9, 2019

Avoid ambiguous signing certificates on macOS.
If a user manually installed a new code signing certificate into the
login keychain as documented, the install script will fail to sign r2
because the sys/macos-cert.sh script generates a new certificate in
the System keychain. This results in the following error when signing:

org.radare.radare2: ambiguous (matches "org.radare.radare2" in /Users/user/Library/Keychains/login.keychain-db and "org.radare.radare2" in /Library/Keychains/System.keychain)

To be constistent with sys/macos-cert.sh, change manual certificate
generation steps to install to System keychain instead of login
keychain.

Mention that code signing certificate generation is automated by the
install script before listing the manual steps to generate a new
code signing certificate.

Avoid ambiguous signing certificates on macOS.
If a user manually installed a new code signing certificate into the
login keychain as documented, the install script will fail to sign r2 
because the `sys/macos-cert.sh` script generates a new certificate in
the System keychain. This results in the following error when signing:
```
org.radare.radare2: ambiguous (matches "org.radare.radare2" in /Users/user/Library/Keychains/login.keychain-db and "org.radare.radare2" in /Library/Keychains/System.keychain)
```
To be constistent with `sys/macos-cert.sh`, change manual certificate
generation steps to install to System keychain instead of login
keychain.

Mention that code signing certificate generation is automated by the
install script before listing the manual steps to generate a new
code signing certificate.
@radare radare merged commit 15d0bb4 into radareorg:master Nov 10, 2019
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.

None yet

2 participants