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

name conflict - electron(atom-shell) #10

Open
romgrk opened this issue Dec 14, 2015 · 5 comments
Open

name conflict - electron(atom-shell) #10

romgrk opened this issue Dec 14, 2015 · 5 comments

Comments

@romgrk
Copy link

romgrk commented Dec 14, 2015

Hello,

I would just like to highlight the fact that this package has a name conflict with https://github.com/atom/electron, and that it might be a good thing to advertise the fact that a name-conflict exists. (executable file is called electron in both packages)

Thanks

@dmr
Copy link

dmr commented Feb 6, 2016

This naming problem is causing a lot of conflicts: electron/electron#3708

@romgrk
Copy link
Author

romgrk commented Feb 6, 2016

@logicalparadox up. This can make you loose quite a lot of time. I won't make the mistake again, but think that every developer that makes the mistake of running npm install electron instead of npm install electron-prebuilt will loose time. Please advertise the fact that a name-conflict exists.

@jacobq
Copy link

jacobq commented Mar 9, 2016

+1; I know it's not your fault that they picked the same name as you were using (looks like your first commit here was in 2012 and the atom shell -> electron rename happened in 2015, I believe), but because their project is far more well-known it can be quite confusing to mistakenly run npm install --save electron (to install atom/electron), have it succeed, but then get unexpected errors further down the line.

@Hypercubed
Copy link

I had the same issue. Installing logicalparadox/matcha (which depends on electron) broke juliangruber/browser-run (which uses electron-prebuilt). To fix this with npm v3.9 I did the following:

npm install matcha
npm uninstall electron
cd node_modules/matcha
npm install

@chocolateboy
Copy link

chocolateboy commented Sep 27, 2016

It's even more confusing now that GitHub's project is called electron on npm (and has been since 2016-07-28).

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

5 participants