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

Atom Framework.framework fails to codesign #171

Closed
probablycorey opened this issue Jan 31, 2014 · 5 comments
Closed

Atom Framework.framework fails to codesign #171

probablycorey opened this issue Jan 31, 2014 · 5 comments

Comments

@probablycorey
Copy link

According to this blog codesigning has changed in Mavericks. So we now need to codesign included frameworks and apps individually.

This shouldn't be a problem Atom Shell needs to worry about because we can codesign the frameworks when Atom is built. For codesigning these work fine:

codesign -f -vvv -s 'Developer ID Application: GitHub' out/Release/Atom\ Helper.app
out/Release/Atom Helper.app: signed bundle with Mach-O thin (x86_64) [com.github.atom.helper]
codesign -f -vvv -s 'Developer ID Application: GitHub' frameworks/Mantle.framework 
frameworks/Mantle.framework: signed bundle with Mach-O thin (x86_64) [com.github.Mantle]

But Atom Framework.framework fails to codesign:

codesign -f -vvv -s 'Developer ID Application: GitHub' out/Release/Atom\ Framework.framework
out/Release/Atom Framework.framework: bundle format unrecognized, invalid, or unsuitable
@zcbenz zcbenz closed this as completed in fe30880 Jan 31, 2014
@zcbenz
Copy link
Member

zcbenz commented Jan 31, 2014

This was caused by lack of Info.plist in Atom Framework.framework, and is fixed now. And you also need to pass --deep to codesign: http://stackoverflow.com/a/17396143/682252.

@probablycorey
Copy link
Author

Awesome, thanks!

@thebyrd
Copy link

thebyrd commented Oct 17, 2014

I upgraded to Yosemite and now this has stopped working. I'm using the most recent atom shell release.

I also found this: http://furbo.org/2013/10/17/code-signing-and-mavericks/

$ codesign --verbose --deep --force --sign "Company, LLC." Atom.app
Atom.app: bundle format is ambiguous (could be app or framework)
In subcomponent: /Users/david/Downloads/Atom.app/Contents/Frameworks/Atom Framework.framework

@zcbenz
Copy link
Member

zcbenz commented Oct 18, 2014

@thebyrd Can you open a new issue for this?

@thebyrd
Copy link

thebyrd commented Oct 18, 2014

Yep, #720

kevinsawicki pushed a commit that referenced this issue May 9, 2017
Call [NSUserNotification setContentImage] conditionally using respondsToSelector
kevinsawicki pushed a commit that referenced this issue May 9, 2017
Call [NSUserNotification setContentImage] conditionally using respondsToSelector
kevinsawicki pushed a commit that referenced this issue Jul 6, 2017
…ckage-rename

Update readme after package rename
gerhardberger pushed a commit to gerhardberger/electron that referenced this issue Dec 26, 2021
gerhardberger pushed a commit to gerhardberger/electron that referenced this issue Dec 26, 2021
This reverts commit e20fcf2095dfe500ebcb429c9a87363331f48b72.

electron-builder still has problems even after the introduction of
the electronBranding configuration options.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants