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

Sign julia.exe executable within the installer #40512

Merged
merged 2 commits into from
Apr 20, 2021
Merged

Sign julia.exe executable within the installer #40512

merged 2 commits into from
Apr 20, 2021

Conversation

musm
Copy link
Contributor

@musm musm commented Apr 17, 2021

Issue: Slack report that the julia.exe executable isn't signed

This patch ends up signing the julia.exe in addition to the installer which we already sign.

Universally adding the sign flag to the existing Source Line ends up signing all executables and dlls, which is slow and probably not necessary. Thus, the exe is filtered away and a second step is added that handles copying the julia executable to the installer and signing it individually.

@musm musm changed the title Add sign for executables within installer Sign julia.exe executable within the installer Apr 17, 2021
@musm musm closed this Apr 19, 2021
@musm musm reopened this Apr 19, 2021
@ViralBShah ViralBShah added the system:windows Affects only Windows label Apr 19, 2021
@ViralBShah
Copy link
Member

Do we want to backport to 1.6?

@ararslan ararslan added the backport 1.6 Change should be backported to release-1.6 label Apr 19, 2021
@musm musm merged commit 592db58 into master Apr 20, 2021
@musm musm deleted the inno-signer branch April 20, 2021 14:40
@KristofferC KristofferC mentioned this pull request May 4, 2021
45 tasks
@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system:windows Affects only Windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants