Skip to content

Commit

Permalink
Just depend on node-uuid, don't include it.
Browse files Browse the repository at this point in the history
Since this isn't required for the npm self-install process, don't bundle
it or load as a git submodule.  It's only necessary for the adduser
command.
  • Loading branch information
isaacs committed Apr 27, 2011
1 parent 4fc175e commit 75b3264
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 89 deletions.
1 change: 1 addition & 0 deletions .gitignore
Expand Up @@ -6,5 +6,6 @@ test/packages/npm-test-depends-on-spark/which-spark.log
test/packages/test-package/random-data.txt
test/root
node_modules/ronn
node_modules/node-uuid
node_modules/.bin
npm-debug.log
3 changes: 0 additions & 3 deletions LICENSE
Expand Up @@ -30,6 +30,3 @@ officially affiliated with Joyent, Inc.
Packages published in the npm registry are not part of npm
itself, and are the sole property of their respective
maintainers.

This software contains the node-uuid library. Copyright 2010
Robert Kieffer. Used under the MIT license.
2 changes: 1 addition & 1 deletion lib/utils/npm-registry-client/adduser.js
@@ -1,7 +1,7 @@

module.exports = adduser

var uuid = require("../uuid")
var uuid = require("node-uuid")
, request = require("./request")
, log = require("../log")
, base64 = require("../base64")
Expand Down
84 changes: 0 additions & 84 deletions lib/utils/uuid.js

This file was deleted.

6 changes: 5 additions & 1 deletion package.json
Expand Up @@ -52,7 +52,11 @@
}
, "main" : "npm"
, "bin" : "./bin/npm.js"
, "dependencies" : { "semver" : "1", "abbrev" : "1", "nopt" : "1" }
, "dependencies" :
{ "semver" : "1"
, "abbrev" : "1"
, "nopt" : "1"
, "node-uuid" : "1.1" }
, "bundleDependencies" : [ "semver", "abbrev", "nopt" ]
, "devDependencies" : { "ronn" : "" }
, "engines" : { "node" : "0.4 || 0.5", "npm" : "1" }
Expand Down

0 comments on commit 75b3264

Please sign in to comment.