search

jackfranklin / pulldown

The minimal JavaScript package manager.
175 stars 7 forks source link

Deal with API errors #91

Closed jackfranklin closed 10 years ago

jackfranklin commented 10 years ago
jackfranklin commented 10 years ago

@phuu

jackfranklin commented 10 years ago

Eurgh, this isn't building because some modules we depend on have dependencies that use the new ^0.2.0 syntax, which Node 0.8 doesn't like. Time to ditch 0.8 support?

jackfranklin commented 10 years ago

@phuu thoughts on above?

jackfranklin commented 10 years ago

I'm confused why this branch fails, because there are none of the ^ dependencies in the package.json.

jackfranklin commented 10 years ago

So one of our dependencies has a ^ hidden in it somewhere:

npm ERR! Error: No compatible version found: minimatch@'^0.3.0'
npm ERR! Valid install targets:
npm ERR! ["0.0.1","0.0.2","0.0.4","0.0.5","0.1.1","0.1.2","0.1.3","0.1.4","0.1.5","0.2.0","0.2.2","0.2.3","0.2.4","0.2.5","0.2.6","0.2.7","0.2.8","0.2.9","0.2.10","0.2.11","0.2.12","0.2.13","0.2.14","0.3.0"]
npm ERR!     at installTargetsError (/home/travis/.nvm/v0.8.26/lib/node_modules/npm/lib/cache.js:719:10)
npm ERR!     at /home/travis/.nvm/v0.8.26/lib/node_modules/npm/lib/cache.js:641:10
npm ERR!     at RegClient.get_ (/home/travis/.nvm/v0.8.26/lib/node_modules/npm/node_modules/npm-registry-client/lib/get.js:101:14)
npm ERR!     at RegClient.<anonymous> (/home/travis/.nvm/v0.8.26/lib/node_modules/npm/node_modules/npm-registry-client/lib/get.js:37:12)
npm ERR!     at fs.readFile (fs.js:176:14)
npm ERR!     at Object.oncomplete (fs.js:297:15)
npm ERR! If you need help, you may report this log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR!     <npm-@googlegroups.com>
npm ERR! System Linux 2.6.32-042stab079.5
npm ERR! command "/home/travis/.nvm/v0.8.26/bin/node" "/home/travis/.nvm/v0.8.26/bin/npm" "install"
npm ERR! cwd /home/travis/build/jackfranklin/pulldown
npm ERR! node -v v0.8.26
npm ERR! npm -v 1.2.30

I hate this.

Not going to merge this because doing so drops 0.8 which seems silly. @phuu any pointers welcome. Screw dependencies.

lloydwatkin commented 10 years ago

If running via travis then a simple:

before_install:
  - npm i -g npm

Will fix this for you :)

lloydwatkin commented 10 years ago

https://github.com/jackfranklin/pulldown/pull/93

jackfranklin commented 10 years ago

@lloydwatkin you might just be my hero.