dodo / node-unicodetable

unicode lookup table
MIT License
42 stars 34 forks source link

0.4.7 Install problems #9

Closed dsyph3r closed 10 years ago

dsyph3r commented 10 years ago

Installing v0.4.7 fails. You can replicate this by simply doing

npm install unicode

Output

fs.js:1440
  var keys = Object.keys(options);
                    ^
TypeError: Object.keys called on non-object
    at Function.keys (native)
    at new fs.WriteStream (fs.js:1440:21)
    at Object.fs.createWriteStream (fs.js:1420:10)
    at newFile ([removed]/node_modules/unicode/install.js:49:19)
    at BufferStream.<anonymous> ([removed]/node_modules/unicode/install.js:74:23)
    at BufferStream.<anonymous> ([removed]/node_modules/unicode/node_modules/bufferstream/lib/buffer-stream.js:175:29)
    at BufferStream.EventEmitter.emit (events.js:91:17)
    at BufferStream.split ([removed]/node_modules/unicode/node_modules/bufferstream/lib/buffer-stream.js:42:12)
    at BufferStream.write ([removed]/node_modules/unicode/node_modules/bufferstream/lib/buffer-stream.js:210:17)
    at BufferStream.__bind [as write] ([removed]/node_modules/unicode/node_modules/bufferstream/lib/buffer-stream.js:3:61)

v0.4.6 installs OK

dodo commented 10 years ago

ah, you're probably using node v0.8, which i forgot to test. fixed with 08f297e4 in 0.4.8

thank you for your report.

dsyph3r commented 10 years ago

Thats correct, running node v0.8.

I can confirm that v0.4.8 installs correctly.

Thanks for fixing so quickly.