diff options
author | isaacs <i@izs.me> | 2011-04-19 04:23:40 +0400 |
---|---|---|
committer | isaacs <i@izs.me> | 2011-04-19 04:49:09 +0400 |
commit | c97d874289abc3d013f3a9c566fc11ce765fbd89 (patch) | |
tree | f3222f4043c547cac1e33a4be01caa518588ec6d | |
parent | 6b33da1a0c9185a7dd204644b0860bf567554c8e (diff) |
adding todo's
-rw-r--r-- | lib/install.js | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/lib/install.js b/lib/install.js index 38aa7c1df..650a9e156 100644 --- a/lib/install.js +++ b/lib/install.js @@ -197,6 +197,10 @@ function install (args, cb) { // just like installMany, but also add the existing packages in // where/node_modules to the previously object. +// +// TODO: If where/package.json exists, and "what" is a package name +// without a specific version, then limit to the version in the +// dependencies hash. function installManyTop (what, where, previously, explicit, cb) { var nm = path.resolve(where, "node_modules") , names = explicit @@ -333,6 +337,9 @@ function checkCycle (target, previously, cb) { // // Solution: Simply flat-out refuse to install any name@version // that is already in the prototype tree of the previously object. + // A more correct, but more complex, solution would be to symlink + // the deeper thing into the new location. + // Will do that if anyone whines about this irl. var p = Object.getPrototypeOf(previously) , name = target.name |