search

meteorhacks / npm

Complete NPM integration for Meteor
http://meteorhacks.com/complete-npm-integration-for-meteor.html
MIT License
508 stars 43 forks source link

Meteor.npmRequire doesn't exist #41

Closed DSpeichert closed 10 years ago

DSpeichert commented 10 years ago 
W20140828-07:17:13.140(2)? (STDERR) Meteor.require is deprecated. Please use Meteor.npmRequire instead!

and

W20140828-07:19:47.733(2)? (STDERR) TypeError: Object #<Object> has no method 'npmRequire'

The "deprecated" one actually works.

arunoda commented 10 years ago

You mean Meteor.npmRequire('connect') is not working? strange!

Arunoda Susiripala I curate Meteor Weekly - Check it out! http://meteorhacks.com/meteor-weekly/?utm_source=email-footer&utm_medium=email&utm_campaign=meteorweekly

On Thu, Aug 28, 2014 at 10:54 AM, Daniel Speichert <notifications@github.com

wrote:

W20140828-07:17:13.140(2)? (STDERR) Meteor.require is deprecated. Please use Meteor.npmRequire instead!

and

W20140828-07:19:47.733(2)? (STDERR) TypeError: Object # has no method 'npmRequire'

The "deprecated" one actually works.

— Reply to this email directly or view it on GitHub https://github.com/meteorhacks/npm/issues/41.

DSpeichert commented 10 years ago

That's exactly what it is.

arunoda commented 10 years ago

Send me a copy of re-produceable repo. I'll dig in.

Arunoda Susiripala

@arunoda http://twitter.com/arunoda http://gplus.to/arunodahttps://github.com/arunoda http://www.linkedin.com/in/arunoda

On Thu, Aug 28, 2014 at 11:10 AM, Daniel Speichert <notifications@github.com

wrote:

That's exactly what it is.

— Reply to this email directly or view it on GitHub https://github.com/meteorhacks/npm/issues/41#issuecomment-53675781.

DSpeichert commented 10 years ago

I'm sorry, I accidentally changed Npm.require to Npm.npmRequire along Meteor.require. Everything is fine with the package. Thanks!