search

dalinaum / IRCCloud

Mac App, Win App and Linux app for IRCCloud. (An Electron wrapper for IRCCloud.)
69 stars 8 forks source link

Remove shell scripts. #5

Open dalinaum opened 9 years ago

dalinaum commented 9 years ago

Make some npm modules to remove shell scripts.

dalinaum commented 9 years ago 
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 commented 9 years ago

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 commented 9 years ago

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

Tux64 commented 9 years ago

@dalinaum 503f9d18e3

dalinaum commented 9 years ago

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