search

bahamas10 / hue-cli

A command line interface to phillips hue
http://bahamas10.github.com/hue-cli/
221 stars 24 forks source link

Arguments to path.join must be strings #4

Closed momo-the-monster closed 10 years ago

momo-the-monster commented 10 years ago

Just installed this globally via npm, everything ok except for the following warning:

npm WARN package.json github-url-from-git@1.1.1 No reposityory field.

Whenever I attempt to run any hue command, I get the following error:

path.js:204
    throw new TypeError('Arguments to path.join must be strings');
TypeError: Arguments to path.join must be strings
    at f (path.js:204:15)
    at Object.filter (native)
    at Object.exports.join (path.js:209:40)
    ...

Running Node.js 0.10.24 on Windows 7 64-bit.

bahamas10 commented 10 years ago

ah, interesting. I've never tested on windows before, but my guess is process.env.HOME is not set... i'll have more time to look at this tonight, but i'm guessing i'll have to find a platform-independent way to gather the home directory.

On Thu, Jan 16, 2014 at 12:58 PM, Momo the Monster <notifications@github.com

wrote:

Just installed this globally via npm, everything ok except for the following warning:

npm WARN package.json github-url-from-git@1.1.1 No reposityory field.

Whenever I attempt to run any hue command, I get the following error:

path.js:204 throw new TypeError('Arguments to path.join must be strings'); TypeError: Arguments to path.join must be strings at f (path.js:204:15) at Object.filter (native) at Object.exports.join (path.js:209:40) ...

Running Node.js 0.10.24 on Windows 7 64-bit.

— Reply to this email directly or view it on GitHubhttps://github.com/bahamas10/hue-cli/issues/4 .

bahamas10 commented 10 years ago

https://github.com/bahamas10/hue-cli/commit/719728ac266917c103ad490c6aa77e951c79c038

try v0.1.12 from npm

momo-the-monster commented 10 years ago

thanks for the update! Unfortunately, 0.0.12 gives me the same error.

bahamas10 commented 10 years ago

hm interesting. I should have access to a windows 7 machine in the next couple of days, i can dig deeper into the bug then..

bahamas10 commented 10 years ago

ha, i'm embarrassed to admit this mistake :p, but try it now @0.0.13

https://github.com/bahamas10/hue-cli/commit/baef890fc94ba8837d9b221c0c14a8a3480a8e57

momo-the-monster commented 10 years ago

It's working!

bahamas10 commented 10 years ago

nice! let me know if you have any other issues, using this on windows in uncharted territory

momo-the-monster commented 10 years ago

Will do! The app and support are much appreciated. I'm getting my head around this API for a fun project and this cli is great for quick debugging.

On Jan 16, 2014, at 5:45 PM, Dave Eddy notifications@github.com wrote:

nice! let me know if you have any other issues, using this on windows in uncharted territory

— Reply to this email directly or view it on GitHub.