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

Feature request: Make the insatller too #5

Open
Altiano opened this issue Jan 19, 2016 · 10 comments
Open

Feature request: Make the insatller too #5

Altiano opened this issue Jan 19, 2016 · 10 comments
Assignees

Comments

@Altiano
Copy link

Altiano commented Jan 19, 2016

why not combine this with electron-builder.. so that we also have the installer..

@vrunoa
Copy link
Member

vrunoa commented Jan 19, 2016

cool idea @Altiano , i'll try to make same implementation next week

@Altiano
Copy link
Author

Altiano commented Jan 19, 2016

great.. I'll be waiting for it 👍

@vrunoa vrunoa self-assigned this Jan 19, 2016
@Cryptix23
Copy link

+1 I would like this option a lot too. !

@dorianpula
Copy link

@vrunoa Any luck with adding in the electron-builder bit?

@develar
Copy link

develar commented Jun 30, 2016

@dorianpula I doubt that is it is required. electron-builder allows you to just use convention, infers a lot of data from package.json, allows to specify configuration in the package.json build. So, such wizard is not required. See https://github.com/develar/onshape-desktop-shell/blob/master/package.json for example.

@dorianpula
Copy link

@develar Ah ok. I'll take a look at the example. I ran into some issues using plain old electron-packager which may be caused by my use of electron-prebuilt-compile (since I'd like to use some of the draft features from es2015). Using epi made things work. Hopefully getting setup in electron-builder will be easier.

@vrunoa
Copy link
Member

vrunoa commented Jun 30, 2016

@dorianpula @develar sorry for the late late reply. I'm really busy with some deadlines I couldn't reach out to even read about electron-builder module. As soon as I can I would give you feedback. PR's are welcomed 😉

@dorianpula
Copy link

Thanks @develar I got electron-builder to work, and it does exactly what I need it to do! I didn't have to do anything fancy, outside of tricky issue with the way the Linux build handles icons.

@vrunoa having an interactive setup to configure electron-builder and update a target package.json would be nice, and it would most likely replace the need for dealing with electron-packager directly.

@develar
Copy link

develar commented Jul 1, 2016

it would most likely replace the need for dealing with electron-packager directly.

if you use electron-builder, you don't need to use electron-packager.

tricky issue with the way the Linux build handles icons.

You can always create plain old folder of icons and don't rely on icns transform.

having an interactive setup to configure electron-builder and update a target package.json would be nice

Any possible scenario, please. Why it is more convenient than just edit your package.json by hand?

@Altiano
Copy link
Author

Altiano commented Jul 1, 2016

@develar
Any possible scenario, please. Why it is more convenient than just edit your package.json by hand?

I think it's just a matter of personal preferences. Some people love interaction more than configuration in some cases.

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

No branches or pull requests

6 participants
@vrunoa @develar @dorianpula @Altiano @Cryptix23 and others