upgrade deps, refactor source handler to work with meteor 1.2

[aaronjudd]

updates to bower 1.5.2
Resolves #97

[mquandalle]

You should also remove this now obsolete comment.

[mquandalle]

Woah, thanks! Now that Meteor 1.2 is out we should merge this ASAP. This looks good to me from a quick review, but I haven't tested the code yet. @aaronjudd Could you comment a little bit more on the changes you have made? @lorensr what do you think of this?

[lorensr]

rm -rf .meteor/local/bower
meteor update
meteor

.meteor/local/bower is not created. When using pull/99/head, packages are correctly downloaded and shipped to client. Don't have a comprehensive suite of bower.jsons though, just my own. Does not regress on the security issue.

[aaronjudd]

@lorensr
I tested with a clean project, with only bower added, created a lib dir, dropped in a pretty big bower.json and it worked without issue (and created .meteor/local/bower). Also have tested within a package (my main use case is installing lib in a package) - and that works. However - I noticed that the directory did not seem to be created with Meteor 1.2.0 - but does work with Meteor 1.2.0.1. Are you testing with an older Meteor release? I would think these changes would be backward compatible, but didn't test.

[lorensr]

I'm on 1.2.0.1 – sorry, your PR works for me – first part of my comment was
just confirming that the currently-published version of bower doesn't work.

On Wed, Sep 23, 2015 at 1:29 AM, Aaron Judd [email protected]
wrote:

@lorensr https://github.com/lorensr
I tested with clean project, with only bower added, created a lib dir,
dropped in a pretty big bower.json and it worked without issue (and created
.meteor/local/bower). Also have tested within a package (my main use case
is installing lib in a package) - and that works. However - I noticed that
the directory did not seem to be created with Meteor 1.2.0 - but does work
with Meteor 1.2.0.1. Are you testing with an older Meteor release? I would
think these changes would be backward compatible, but didn't test.

—
Reply to this email directly or view it on GitHub
#99 (comment)
.

[aaronjudd]

Ah- great. Made me do some extra testing though, so not a bad thing.

[aaronjudd]

@lorensr @mquandalle merge and publish?

[mquandalle]

Done, thank you very much :-)

[lscordilis]

Thanks very much for the quick fix, guys. I think I speak for the community when I say Meteor-bower is still a great and popular tool and we really appreciate the continued support.

[allanlundhansen]

Great fast work thank you!