Skip to content
This repository has been archived by the owner on Mar 19, 2021. It is now read-only.

Remove shell scripts. #5

Open
dalinaum opened this issue Jun 26, 2015 · 5 comments
Open

Remove shell scripts. #5

dalinaum opened this issue Jun 26, 2015 · 5 comments

Comments

@dalinaum
Copy link
Owner

Make some npm modules to remove shell scripts.

@dalinaum
Copy link
Owner Author

electron-prebuilt@0.28.3 postinstall /home/sjpark/IRCCloud/node_modules/electron-prebuilt
> node install.js

sh: 1: node: not found
npm WARN This failure might be due to the use of legacy binary "node"
npm WARN For further explanations, please read
/usr/share/doc/nodejs/README.Debian

npm ERR! weird error 127
npm ERR! not ok code 0

electron-prebuilt doesn't run on Linux. I have to find more generic solution.

@Tux64
Copy link
Collaborator

Tux64 commented Aug 26, 2015

Note from /usr/share/doc/nodejs/README.Debian:

The upstream name for the Node.js interpreter command is "node".
In Debian the interpreter command has been changed to "nodejs".

This was done to prevent a namespace collision: other commands use
the same name in their upstreams, such as ax25-node from the "node"
package.

Scripts calling Node.js as a shell command must be changed to instead
use the "nodejs" command.

Sym-linking /usr/bin/nodejs to /usr/bin/node fixes the issue.

@dalinaum
Copy link
Owner Author

Thanks. I had a mistake. But I still want to remove shell scripts.

@Tux64
Copy link
Collaborator

Tux64 commented Sep 9, 2015

@dalinaum 503f9d1

@dalinaum
Copy link
Owner Author

dalinaum commented Sep 9, 2015

@Tux64 LGTM. I think you can imporve this app. So I add you as a collobrator. You can push your commits directly.

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

No branches or pull requests

2 participants