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 signature removal from macos universal binaries. #14978

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

aarlt
Copy link
Member

@aarlt aarlt commented Apr 3, 2024

No description provided.

@r0qs
Copy link
Member

r0qs commented Apr 3, 2024

Out of curiosity, why this was added in the first place?

@nikola-matic
Copy link
Collaborator

This was the issue with our latest macos release, right? The binaries were completely unsigned, and could not be run?

@aarlt
Copy link
Member Author

aarlt commented Apr 3, 2024

Out of curiosity, why this was added in the first place?

It looked like that binaries can just be run if there were not signed - because we never had any signature before. However, this was only partly true, the execution of code needs at least an ad-hoc signature on apple silicon where on x86 it is not really needed (at least for x86 non-universal binaries). Sadly I didn't really tested it on my local machine before the last release and only noticed that the executable cannot run without any signature on apple silicon after.

This was the issue with our latest macos release, right? The binaries were completely unsigned, and could not be run?

Yes, exactly. Completely unsigned executables cannot be run on apple silicon. The problem was mainly that I was not testing the resulting binary on a different machine. The tests on macOS where executed correctly, because there seem to be some magic involved if a signature was removed from an executable on the same machine.

However, that means we need at least that ad-hoc signature. Probably we need to discuss next meeting whether we want to sign our executables with a proper certificate (we would need an Apple ID for that), or that we just use it with that ad-hoc signature.

@cameel cameel force-pushed the remove_remove_signature_from_universal_binary branch from ec8ff4c to bcb10f8 Compare April 4, 2024 18:52
@cameel
Copy link
Member

cameel commented Apr 4, 2024

There would probably be fewer questions if you simply linked this PR to the previous issues/PRs :) For reference, this is the original issue: #14813. And this is the PR that added signature removal: #14869.

Probably we need to discuss next meeting whether we want to sign our executables with a proper certificate (we would need an Apple ID for that), or that we just use it with that ad-hoc signature.

In that case we should reopen the issue.

@cameel cameel merged commit 592e429 into develop Apr 8, 2024
73 checks passed
@cameel cameel deleted the remove_remove_signature_from_universal_binary branch April 8, 2024 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants