mquandalle / meteor-bower

[DEPRECATED] Use bower packages in your Meteor app
http://bower.io/search/
MIT License
157 stars 28 forks source link

Fails silently in Meteor 1.2+ #97

Closed cee-dub closed 9 years ago

cee-dub commented 9 years ago

Spent a few hours unable to understand where bower packages were supposed to end up automatically installed during the meteor build process, and it turned out to be due to our usage of meteor's 1.2 RC and the new compiler API.

Could we update the README with a warning, or is there any way we can help #94 along?

mquandalle commented 9 years ago

I would merge a PR adding a warning in the README.

jordangarside commented 9 years ago

:+1:

lscordilis commented 9 years ago

Yeah, none of my bower dependencies seem to be loaded to the browser since I updated to Meteor 1.2. I will need to rollback to the previous Meteor build because angular crashes when it can't find the dependencies I'm trying to load with the $injector, thus rendering the all the development builds of all my angular projects useless. +1

aaronjudd commented 9 years ago

For me, this works: In handler.js, remove the "bower.json" param in registrySourceHandler, and only having single handler for "json". I'm digging deeper to see if what the ramifications of that are, but at least it's a clue.