`npm install -g tty.js` doesn't work as expected about tty.js

[chjj]

Sorry for the obvious question, but does /Users/lib/node_modules/tty.js/index.js exist? That's really strange.

[TooTallNate]

Negatory. Note that it's looking in /Users for some reason. When really it should be looking in /usr/local/lib/node_modules:

☮ ~ (master) ⚡ which tty.js
/usr/local/bin/tty.js

☮ ~ (master) ⚡ ls -l /usr/local/lib/node_modules/tty.js/
total 16K
-rw-r--r-- 1 nrajlich wheel 1.1K Feb  3 00:01 LICENSE
-rw-r--r-- 1 nrajlich wheel 3.2K Apr  4 03:18 README.md
drwxr-xr-x 3 nrajlich wheel  102 Apr  9 08:02 bin/
-rw-r--r-- 1 nrajlich wheel   42 Feb  2 09:48 index.js
drwxr-xr-x 4 nrajlich wheel  136 Apr  9 08:02 lib/
drwxr-xr-x 3 nrajlich wheel  102 Apr  9 08:02 man/
drwxr-xr-x 6 nrajlich wheel  204 Apr  9 08:02 node_modules/
-rw-r--r-- 1 nrajlich wheel 1.1K Apr  9 08:02 package.json
drwxr-xr-x 7 nrajlich wheel  238 Apr  9 08:02 static/

Try it yourself, I'm sure it'll happen for you too.

[TooTallNate]

Seems like you're not using dirname(1) properly. This patch makes things work for me:

diff --git a/bin/tty.js b/bin/tty.js
index d824a74..9f8f841 100755
--- a/bin/tty.js
+++ b/bin/tty.js
@@ -4,7 +4,9 @@ f=$0
 while test -L "$f"; do
   f=$(readlink "$f")
 done
-dir="$(dirname "$f")/.."
+dir=`dirname "$f"`
+dir=`dirname "$dir"`
+dir=`readlink -f "$dir"`

 node=$(which node 2>/dev/null)
 if test -z "$node"; then

[chjj]

Oh, I think I figured out the problem. My readlink resolution doesn't take into account relative symlinks (I can't use readlink -f there because it's not portable), so if it hits a link that's relative, it won't be able to resolve the path correctly and end up giving you a weird one instead.

[lucaspiller]

@TooTallNate Unfortunately this doesn't work on Mac OS X as the included version of readlink doesn't support the -f flag. I'm not sure of a better solution, I came across this, but it doesn't seem great: http://serverfault.com/questions/40144/how-can-i-retrieve-the-absolute-filename-in-a-shell-script-on-mac-os-x

[TooTallNate]

@chjj If I were you I would just rewrite this startup script in node; would make things a lot simpler.

[chjj]

@TooTallNate, I originally chose a bash script because it's easier to daemonize the process. I'll rewrite it in node though.

[chjj]

Fixed for next release. The start script was rewritten in node.