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

macOS codesigning: only attempt if $MACOS_CODESIGN_IDENTITY is set #33228

Merged
merged 1 commit into from
Sep 16, 2019

Conversation

staticfloat
Copy link
Member

This has the following advantages:

  • It allows our buildbots to pass in which code signing identity they
    want to use (rather than having the identity hash hardcoded in our build
    system)

  • It stops blindly attempting to codesign on random user's machines

  • It causes codesign failure to stop the build

@staticfloat staticfloat added backport 1.0 building Build system, or building Julia or its dependencies system:mac Affects only macOS labels Sep 11, 2019
@ViralBShah
Copy link
Member

Will address #33141 in the future

@staticfloat staticfloat force-pushed the sf/parameterized_codesigning branch 2 times, most recently from 8b89956 to c899e9d Compare September 16, 2019 08:13
This has the following advantages:

* It allows our buildbots to pass in which code signing identity they
want to use (rather than having the identity hash hardcoded in our build
system)

* It stops blindly attempting to codesign on random user's machines

* It causes codesign failure to stop the build
@staticfloat staticfloat force-pushed the sf/parameterized_codesigning branch from c899e9d to 1ddcd23 Compare September 16, 2019 18:09
@staticfloat staticfloat merged commit 0760842 into master Sep 16, 2019
@delete-merged-branch delete-merged-branch bot deleted the sf/parameterized_codesigning branch September 16, 2019 18:09
KristofferC pushed a commit that referenced this pull request Sep 18, 2019
…33228)

This has the following advantages:

* It allows our buildbots to pass in which code signing identity they
want to use (rather than having the identity hash hardcoded in our build
system)

* It stops blindly attempting to codesign on random user's machines

* It causes codesign failure to stop the build

(cherry picked from commit 0760842)
@KristofferC KristofferC mentioned this pull request Sep 18, 2019
18 tasks
KristofferC pushed a commit that referenced this pull request Sep 19, 2019
…33228)

This has the following advantages:

* It allows our buildbots to pass in which code signing identity they
want to use (rather than having the identity hash hardcoded in our build
system)

* It stops blindly attempting to codesign on random user's machines

* It causes codesign failure to stop the build

(cherry picked from commit 0760842)
KristofferC pushed a commit that referenced this pull request Dec 3, 2019
…33228)

This has the following advantages:

* It allows our buildbots to pass in which code signing identity they
want to use (rather than having the identity hash hardcoded in our build
system)

* It stops blindly attempting to codesign on random user's machines

* It causes codesign failure to stop the build

(cherry picked from commit 0760842)
@KristofferC KristofferC mentioned this pull request Dec 3, 2019
56 tasks
KristofferC pushed a commit that referenced this pull request Feb 20, 2020
…33228)

This has the following advantages:

* It allows our buildbots to pass in which code signing identity they
want to use (rather than having the identity hash hardcoded in our build
system)

* It stops blindly attempting to codesign on random user's machines

* It causes codesign failure to stop the build

(cherry picked from commit 0760842)
KristofferC pushed a commit that referenced this pull request Feb 20, 2020
…33228)

This has the following advantages:

* It allows our buildbots to pass in which code signing identity they
want to use (rather than having the identity hash hardcoded in our build
system)

* It stops blindly attempting to codesign on random user's machines

* It causes codesign failure to stop the build

(cherry picked from commit 0760842)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies system:mac Affects only macOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants