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

tools/mac apps won't open #691

Closed
aizuchi0 opened this Issue Dec 12, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@aizuchi0

aizuchi0 commented Dec 12, 2017

After building, the Mac helper apps all launch except ConfigEditor and CmdSequence. From the Finder all one sees is 'The application “CmdSequence” can’t be opened.', but from the CLI:
open ConfigEditor.app/
LSOpenURLsWithRole() failed with error -10810 for the file /Users/xxxxx/Desktop/cosmosdemo/tools/mac/ConfigEditor.app.
LSOpenURLsWithRole() failed with error -10810 for the file /Users/xxxxx/Desktop/cosmosdemo/tools/mac/CmdSequence.app.

@jasonatball

This comment has been minimized.

Collaborator

jasonatball commented Dec 13, 2017

Hmm, I copied an existing Mac app to create the new ones for these two new tools. There's a discussion about this error here: electron-userland/electron-packager#323.

@ryanatball how were the original Mac apps built?

@jasonatball jasonatball self-assigned this Dec 18, 2017

@jasonatball jasonatball added the bug label Dec 18, 2017

@jasonatball

This comment has been minimized.

Collaborator

jasonatball commented Dec 18, 2017

@aizuchi0 I think the only thing missing was the execute privileges on the main.sh shell script. Can you try this and verify it works:

chmod +x /Users/xxxxx/Desktop/cosmosdemo/tools/mac/ConfigEditor.app/Contents/MacOS/main.sh
chmod +x /Users/xxxxx/Desktop/cosmosdemo/tools/mac/CmdSequence.app/Contents/MacOS/main.sh
@aizuchi0

This comment has been minimized.

aizuchi0 commented Dec 18, 2017

@jasonatball That was indeed it. My apologies for not trying to debug that myself, but I'm new to Ruby and didn't want to break anything. Thanks much for the fix!

@jasonatball jasonatball referenced this issue Dec 18, 2017

Merged

Fix install #695

@ryanatball ryanatball added this to the v4.2.2 milestone Apr 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment