Add extra step in documentation for extra symlink inside socket_io

Here's the patch

Sure.

I can make the extra directory in the project, Should we put the "optional" symlinks steps in this new socket_io/README.txt?

The two places in the code that I can see that maybe should be changed are in nodejs.module:

line 100
drupal_add_js(drupal_get_path('module', 'nodejs') . '/socket_io/support/socket.io-client/socket.io.js', array('type' => 'file'));

line 386

$defaults = array(
    'scheme' => variable_get('nodejs_server_scheme', 'http'),
    'secure' => variable_get('nodejs_server_scheme', 'http') == 'https' ? 1 : 0,
    'host' => variable_get('nodejs_server_host', 'localhost'),
    'port' => variable_get('nodejs_server_port', '8080'),
    'resource' => '/node.js/realtime',
    'authkey' => md5(session_id()),
    'serviceKey' => '',
    'websocketSwfLocation' => base_path() . drupal_get_path('module', 'nodejs') . '/socket_io/support/socket.io-client/lib/vendor/web-socket-js/WebSocketMain.swf',
  );

Those are the only places that should change when the folder setup it figured out. Correct?

Speaking of the folder setup, Are you recommending we install socket_io in a non default directory? Or try and boil down the setup to making several symlinks in the new socket_io directory?

Can we change this step in the README.TXT:

> ln -s /path/to/socket/io/install/socket.io /path/to/your/nodejs/module/socket_io/

to

> ln -s /path/to/socket/io/install/socket.io socket.io

I created the symlink with the first command mentioned above and it made the symlink name "package" inside the socket_io directory you just created. This will cause the module to not find the socket javascript file because in line 100 of the module:
drupal_add_js(drupal_get_path('module', 'nodejs') . '/socket_io/socket.io/support/socket.io-client/socket.io.js', array('type' => 'file'));
It's looking in socket_io/socket.io.

For my personal instance, the command was:

ln -s /usr/local/lib/node/.npm/socket.io/active/package/ socket.io

Just to ensure the name of the symlink was "socket.io" and not "package".

cool, I was going to suggest changing the path in the module on line 100 as an alternative, but this works just as well!

I think this step should be changed to just, "Look in the socket_io directory and follow the README instructions there to symlink to your local socket.io path" ... something along those lines, Justin wrote some pretty clear symlinking directions in there.

Sounds good, post the pizzatch and we will rizzeview

Why did you get rid of this step?

-  3. Install npm module
-curl http://npmjs.org/install.sh | sudo sh