anaisbetts / grunt-build-atom-shell

Grunt task to build Electron and rebuild node modules
MIT License
122 stars 11 forks source link

Question: is this repo still the 'official' way to rename electron? #33

Open rishson opened 9 years ago

rishson commented 9 years ago

Not that I don't love this repo - (thanks @paulcbetts ) Its been really useful over the last couple of months, but given the stated need for this repo is to be able to rename the electron .exe on Windows, does electron 0.23.0 fix the original issue (atom/atom-shell#713), given the Windows release notes for 0.23.0 state:

"__Renaming atom.exe no longer breaks native modules."

Is it possible to simply do:

npm i electron-prebuilt

and then:

If you no longer need to build from src, then could this repo move forward with additional tasks that could:

Maybe these tasks belong in electron starter instead? Happy to help contribute grunt tasks if this is the correct assumption and direction.

Rameshv commented 9 years ago

+1

anaisbetts commented 9 years ago

Yep, you're mostly right, that was the main motivation for this package. That being said, this package still has its uses because it makes it super easy to run on non-released branches (either your own fork or being able to run on a specific commit of Electron). However, for most people getting started, grunt-download-atom-shell is sufficient.

@rishson can you send a PR to switch electron-starter to grunt-download-atom-shell? It should be fairly easy because I intentionally kept the parameters very similar

maxkorp commented 9 years ago

I definitely like the idea of having the task handle some of that other stuff. I have manual tasks for doing a lot of that, having a single task (or at least set of tasks in a package) would be awesome.

maxkorp commented 9 years ago

Just wanted to throw out there that I'd be down to help with that, if it's deemed the way you wanna take it (Although I can also understand wanting to keep this for building, in case people want that control over things still or if there are other building benefits (e.g. the correct name in the task manager in windows, etc).

anaisbetts commented 9 years ago

Also, if you're mostly using the electron-prebuilt package, https://github.com/paulcbetts/electron-rebuild will do the native module rebuilding bits of grunt-build-atom-shell. I'm working on the rest :)

iamdriz commented 9 years ago

@paulcbetts doesn't electron-prebuilt just run the app? It doesn't actually create any compiled files or anything. Plus doesn't electron-packager do all of this now? Thanks.

anaisbetts commented 9 years ago

doesn't electron-prebuilt just run the app? It doesn't actually create any compiled files or anything. Plus doesn't electron-packager do all of this now? Thanks.

You still need to rebuild the node native modules you're using against the version of Electron you're using, or else you'll have Problems(tm). That's what electron-rebuild does.

matthiasg commented 9 years ago

how can i find out which version of node is to be used ? e.g when looking at its not declaring which version of node it uses ... https://github.com/atom/electron/releases/tag/v0.35.0

anaisbetts commented 9 years ago

@matthiasg Just use electron-rebuild and you won't have to know that

matthiasg commented 9 years ago

@paulcbetts will try thanks

maxkorp commented 9 years ago

Can confirm, electron rebuild works well. There are a few modules around for renaming your copy of electron prebuilt, cant think of the names of any right now, but I haven't been too satisfied with any of them (theyre all much larger in scope than just that), so I may make one that just renames electron-prebuilt