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

npm run scripts and dependencies for building OSX dist #6

Closed
wants to merge 10 commits into from
Closed

npm run scripts and dependencies for building OSX dist #6

wants to merge 10 commits into from

Conversation

dmison
Copy link

@dmison dmison commented Jan 29, 2016

includes support for building .icns file from SVG as required for OSX build for correct icon

I also added support for rebuilding icon-128.png from the SVG as well because it was there. :-)

The resulting .app is about a 3rd the size of the current download

@dmison
Copy link
Author

dmison commented Jan 29, 2016

urrgh . stupid. I rebased my local and pushed it back to github hence extra commits :-(

@dmison
Copy link
Author

dmison commented Jan 29, 2016

redo

@dmison dmison closed this Jan 29, 2016
@erkyrath
Copy link
Owner

Thanks. I will look at this next week. electron-packager was already on my radar.

When you say the app is smaller, do you know what's omitted?

@dmison
Copy link
Author

dmison commented Jan 29, 2016

it symlinks a few things instead of multiple copies, eg. Electron Framework.framwork/Versions/Current to Electron Framework.framwork/Versions/A.

Looking now I'm wondering if the fact that they weren't symlinked in your download might have been a result of the filecopy in makedist.py because they are definitely symlinked in electron-prebuilt

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

Successfully merging this pull request may close these issues.

2 participants