infer packagejson dependencies from peerDependencies

diff --git a/scripts/lib/build_all.js b/scripts/lib/build_all.js
index 6e02709..d620c38 100644 (file)
--- a/scripts/lib/build_all.js
+++ b/scripts/lib/build_all.js
@@ -57,6 +57,7 @@ var _ = require('underscore'),
       },
       function (done) {
         if (specs[0].npmDev) {
+          done();
           return;
         }
         // step 1: npm install extension if necessary

diff --git a/scripts/lib/util/process_manifest.js b/scripts/lib/util/process_manifest.js
index 90f6030..889d18a 100644 (file)
--- a/scripts/lib/util/process_manifest.js
+++ b/scripts/lib/util/process_manifest.js
@@ -25,8 +25,10 @@ regexp:true, undef:true, strict:true, trailing:true, white:true */
     registerSql = registerSql + "select xt.grant_role_ext('ADMIN', '%@');\n"
       .f(options.name);
 
-    // TODO: infer dependencies from package.json using peerDependencies
-    var dependencies = options.dependencies || [];
+    // infer dependencies from package.json -> peerDependencies
+    // infer dependencies from manifest.js -> dependencies
+    var isPackageJson = !!options.engines; // XXX this is a pretty rough proxy
+    var dependencies = (isPackageJson ? options.peerDependencies : options.dependencies) || [];
     _.each(dependencies, function (dependency) {
       var dependencySql = "select xt.register_extension_dependency('%@', '%@');\n"
           .f(options.name, dependency),