search

fstiewitz / build-tools-cpp

Build your projects in atom
https://atom.io/packages/build-tools
MIT License
31 stars 6 forks source link

Uncaught TypeError: Arguments to path.join must be strings #4

Closed stsdc closed 9 years ago

stsdc commented 9 years ago

Some time after installing and working very good it throws Uncaught TypeError: Arguments to path.join must be strings in devtools.

fstiewitz commented 9 years ago

Can you post a stack trace because I cannot reproduce the error ?

stsdc commented 9 years ago

As soon as I'll get this error again, because I reinstalled the package and it's working again.

stsdc commented 9 years ago

Uncaught TypeError: Arguments to path.join must be strings path.js:384 exports.join path.js:384module.exports.getWD build-parser.coffee:6 module.exports.spawn build-tools-cpp.coffee:100 module.exports.step2 build-tools-cpp.coffee:148 (anonymous function) build-tools-cpp.coffee:21 handler /usr/share/atom/resources/app/src/space-pen-extensions.js:107 jQuery.event.dispatch /usr/share/atom/resources/app/node_modules/space-pen/vendor/jquery.js:4681 elemData.handle /usr/share/atom/resources/app/node_modules/space-pen/vendor/jquery.js:4359 module.exports.CommandRegistry.handleCommandEvent /usr/share/atom/resources/app/src/command-registry.js:253 (anonymous function) /usr/share/atom/resources/app/src/command-registry.js:3 module.exports.KeymapManager.dispatchCommandEvent /usr/share/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:544 module.exports.KeymapManager.handleKeyboardEvent /usr/share/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:386 module.exports.WindowEventHandler.onKeydown /usr/share/atom/resources/app/src/window-event-handler.js:151

It happened after reopening of the editor. All settings of package were cleared.

fstiewitz commented 9 years ago

I can reproduce the error when I open atom outside a project, so that atom.project.getPath() returns undefined. Tomorrow I will add a few checks to make sure path.join only uses strings. Hopefully this solves the problem.

stsdc commented 9 years ago

Thank You.