Skip to content

Commit

Permalink
Merge pull request #9102 from fxcoudert/sign
Browse files Browse the repository at this point in the history
keg: add codesigning
  • Loading branch information
claui authored Nov 14, 2020
2 parents c5fea1b + 4665bb8 commit e945b1c
Showing 1 changed file with 35 additions and 0 deletions.
35 changes: 35 additions & 0 deletions Library/Homebrew/os/mac/keg.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ def change_dylib_id(id, file)
@require_relocation = true
odebug "Changing dylib ID of #{file}\n from #{file.dylib_id}\n to #{id}"
MachO::Tools.change_dylib_id(file, id, strict: false)
apply_ad_hoc_signature(file)
rescue MachO::MachOError
onoe <<~EOS
Failed changing dylib ID of #{file}
Expand All @@ -23,6 +24,7 @@ def change_install_name(old, new, file)
@require_relocation = true
odebug "Changing install name in #{file}\n from #{old}\n to #{new}"
MachO::Tools.change_install_name(file, old, new, strict: false)
apply_ad_hoc_signature(file)
rescue MachO::MachOError
onoe <<~EOS
Failed changing install name in #{file}
Expand All @@ -31,4 +33,37 @@ def change_install_name(old, new, file)
EOS
raise
end

def apply_ad_hoc_signature(file)
return if MacOS.version < :big_sur
return unless Hardware::CPU.arm?

odebug "Codesigning #{file}"
# Use quiet_system to squash notifications about resigning binaries
# which already have valid signatures.
return if quiet_system("codesign", "--sign", "-", "--force",
"--preserve-metadata=entitlements,requirements,flags,runtime",
file)

# If the codesigning fails, it may be a bug in Apple's codesign utility
# A known workaround is to copy the file to another inode, then move it back
# erasing the previous file. Then sign again.
#
# TODO: remove this once the bug in Apple's codesign utility is fixed
Dir::Tmpname.create("workaround") do |tmppath|
FileUtils.cp file, tmppath
FileUtils.mv tmppath, file, force: true
end

# Try signing again
odebug "Codesigning (2nd try) #{file}"
return if quiet_system("codesign", "--sign", "-", "--force",
"--preserve-metadata=entitlements,requirements,flags,runtime",
file)

# If it fails again, error out
onoe <<~EOS
Failed applying an ad-hoc signature to #{file}
EOS
end
end

0 comments on commit e945b1c

Please sign in to comment.