Permalink
Browse files

deployment fix for issue caused by npm/npm#2195

  • Loading branch information...
1 parent 68974db commit a6966a83c061ac8c2314fa3b7a0d1fbb8deec404 @jstewmon jstewmon committed Feb 24, 2012
Showing with 13 additions and 2 deletions.
  1. +4 −0 .npmignore
  2. +9 −2 package.json
View
@@ -0,0 +1,4 @@
+deps
+dist
+test
+nodelint.cfg
View
@@ -13,9 +13,16 @@
, "url" : "http://github.com/caolan/async/raw/master/LICENSE"
}
]
+, "dependencies":
+ { "uglify-js": "1.2.x"
+ }
, "devDependencies":
- { "uglify-js": ">0.0.0"
- , "nodeunit": ">0.0.0"
+ { "nodeunit": ">0.0.0"
, "nodelint": ">0.0.0"
}
+, "scripts":
+ { "preinstall": "make clean"
+ , "install": "make build"
+ , "test": "make test"
+ }
@caolan
caolan Feb 27, 2012 owner

This is breaking projects without uglifyjs installed globally or at the async/node_modules level, I'm going to have to remove it for now and if you need async.min.js you'll have to manage that dependency manually.

@jstewmon
jstewmon Feb 27, 2012

That's why I moved uglify-js from devDependencies to dependencies. It now gets installed when async is installed (or updated), so it will always be in async/node_modules.

@caolan
caolan Feb 27, 2012 owner

not if the parent project already has uglify-js it in it's dependencies

@jstewmon
jstewmon Feb 27, 2012

oh, snap. you're right. sorry.

This will correct that: jstewmon/async@7d04cf1

}

0 comments on commit a6966a8

Please sign in to comment.