TypeError: "cwd" must be a string on Node 8

[zipang]

After upgrading to Node 8, we have an error on startup when launching a NodeBB server.
This issue has been documented here : https://community.nodebb.org/topic/10765/error-starting-nodebb-after-upgrade

And here is the full error stack which has its origin in the daemon module used by NodeBB :

TypeError: "cwd" must be a string
    at normalizeSpawnArguments (child_process.js:380:11)
    at Object.exports.spawn (child_process.js:465:38)
    at Function.module.exports.daemon (/www/node-bb/node_modules/daemon/index.js:50:31)
    at module.exports (/www/node-bb/node_modules/daemon/index.js:25:20)
    at /www/node-bb/loader.js:223:21
    at FSReqWrap.oncomplete (fs.js:135:15)

According to the API documentation, process.cwd() is effectively a method : https://nodejs.org/api/process.html#process_process_cwd

The fix is quite trivial and i will provide a Pull Request for it.

[zipang]

Is there anyone who could review my PR ? It should be really fast i believe.

[unilynx]

For anyone else affected, a quick workaround is to apply zipang's PR:

npm install 'github:zipang/daemon.node#48d0977c26fb3a6a44ae99aae3471b9d5a761085'

[jhuckaby]

Paging @indexzero, is it possible for you to merge this PR? It's a trivial bug fix, and would help us out. Node 8 is now in LTS, so many more people will be using this. Two character fix:

zipang@48d0977

Before:

var cwd = opt.cwd || process.cwd;

After:

var cwd = opt.cwd || process.cwd();

Thanks!

[p-salido]

@indexzero Ping, can we get #42 into an NPM release?

[p-salido]

The workaround for this issue should be to pass in cwd in options, which typically would be set to /.

[c1tt1]

same issue here but when implemented in modern node way it works