-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
package.json shows "mongodb": "1.3.7-meteor" #17
Comments
@BretFisher Thanks for this. I had the same problem 'demeteorizing' a more-or-less custom app (it's from the 'Discover Meteor' book - Chapter 11). Your solution resolved my issue. |
I am seeing this too, but only since I upgraded Meteor to 0.6.4.1. If I downgrade to 0.6.4, it works again. |
The Meteor guys fixed a security whole in the MongoDB node driver. They forked the driver and created a new version called 1.3.7-meteor, which is not available in npm. I'm looking now to see if I can point the dependency to the github repo to install from there instead of npm. |
The bug has been fixed in newer versions. What I'll do it set the dependency to "1.3.x", which will just grab the latest version 1.3 version of the driver. |
Fixed in 0.2.4 of demeteorizer. |
background
I'm trying to use demeteorizer with codeship.io, which doesn't give me sudo (to install meteor) so I have to use
git clone https://github.com/meteor/meteor.git ~/meteor
to use meteor, which causes a few things to act differently duringbundle
.First run of demeteorizer gives me this:
issue
I'm assuming it's the dev_bundle dependency kit which causes
npm install
(post bundle) to fail, which is somehow picking up the mongodb version with "1.3.7-meteor".This is what demeteorizer puts in package.json
Once that's in the package.json any
npm install
will fail.resolution
not sure what the core issue is: faulty meteor, faulty demeteorizer, etc.
workaround
Workaround is to use sed to remove
-meteor
afterward:sed -i.bak s/-meteor// package.json
The text was updated successfully, but these errors were encountered: