search

eugeneware / debowerify

A browserify transform to enable the easy use of bower components in browserify client javascript projects. This can be used in conjunction with deamdify to require AMD components from bower as well.
493 stars 51 forks source link

Debowerify breaks if given no options? #63

Closed ashrewdmint closed 9 years ago

ashrewdmint commented 9 years ago

When trying to run $ browserify -t debowerify path/to/files/*.js -o path/to/output.js I get this:

/.../node_modules/debowerify/index.js:79
    var bowerOptions = options.bowerOptions || {};
                              ^
TypeError: Cannot read property 'bowerOptions' of undefined

If I make these changes to debowerify's index.js, it works:

8 | module.exports = function (file, options) {
9 |   if (!options) options = {};

I have upgraded to the latest version of node and npm but it still doesn't seem to work as-is.

sukima commented 9 years ago

Have you considered sending the change you made as a pull request?

ashrewdmint commented 9 years ago

OK, here is a pull request with the change mentioned above.

eugeneware commented 9 years ago

Should be fixed by #64. Thanks!