Skip to content

Conversation

fanquake
Copy link
Member

Signature application is now done with signapple.

# Apply detached codesignatures to dist/ (in-place)
signapple apply dist/Bitcoin-Qt.app codesignatures/osx/dist

Signature application is now done with signapple.
@laanwj
Copy link
Member

laanwj commented Mar 17, 2022

ACK 979271a
Checked that the script is unreferenced.

@gruve-p
Copy link
Contributor

gruve-p commented Mar 17, 2022

ACK 979271a

@achow101
Copy link
Member

ACK 979271a

@DrahtBot
Copy link
Contributor

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #24031 (build: don't compress macOS DMG by fanquake)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 979271a, I have reviewed the code and it looks OK, I agree it can be merged.

Call sites of detached-sig-apply.sh were removed in ab9c342 (#22418).

@fanquake fanquake merged commit a17df4e into bitcoin:master Mar 18, 2022
@fanquake fanquake deleted the unused_detached_sig_apply branch March 18, 2022 09:54
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 18, 2022
979271a macdeploy: remove unused detached-sig-apply (fanquake)

Pull request description:

  Signature application is now done with signapple.

  https://github.com/bitcoin/bitcoin/blob/8435d7f11a89bb3f93306646f62cc2179693e072/contrib/guix/libexec/codesign.sh#L84-L85

ACKs for top commit:
  laanwj:
    ACK 979271a
  gruve-p:
    ACK bitcoin@979271a
  achow101:
    ACK 979271a
  hebasto:
    ACK 979271a, I have reviewed the code and it looks OK, I agree it can be merged.

Tree-SHA512: ab51a609d00cead4f33bcfc5b5ff1008ee02363ab1f4c4bf9544631069c237bfa92eac4dfa231bff8a1d702bda6cc92b4151361f74f58e77b595e0cb82a8391a
@bitcoin bitcoin locked and limited conversation to collaborators Mar 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants