add support for building package in dependencies, Fix #883

bin/spm-build

@@ -19,6 +19,7 @@ commander
   .option('--ignore [ignore]', 'determine which id will not be transported')
   .option('--skip [skip]', 'determine which id will not be parsed when analyse')
   .option('--idleading [idleading]', 'prefix of module name, default: {{name}}/{{version}}')
+  .option('--with-deps', 'build package in dependencies')
   .option('--zip', 'archive by zip')
   .parse(process.argv);
 
@@ -46,6 +47,7 @@ var args = {
   idleading: commander.idleading,
   zip: commander.zip,
   force: commander.force,
+  withDeps: commander.withDeps,
   install: true
 };
 

lib/build.js

@@ -70,6 +70,17 @@ function build(args, cb) {
   // check duplicate pkgs
   checkDuplicate(files, pkg);
 
+  // build package in dependencies
+  if (args.withDeps) {
+    var depPkgs = pkg.getPackages();
+    for (var id in depPkgs) {
+      files = files.concat(getFiles(depPkgs[id]).map(function(f) {
+        return 'sea-modules/'+id.replace('@','/')+'/' + f;
+      }));
+    }
+    log.info('withDeps', 'files: ' + files);
+  }
+
   // define task
   defineCleanTask(args);
   defineBuildTask(files, args);

tests/build.js

@@ -243,6 +243,19 @@ describe('build', function() {
     });
   });
 
+  it('js package with deps', function(done) {
+    var opt = {
+      cwd: join(base, 'build-js'),
+      dest: dest,
+      withDeps: true
+    };
+    build(opt, function(err) {
+      should.not.exist(err);
+      assets('build-js-with-deps', dest);
+      done();
+    });
+  });
+
   function assets(prefix, dest) {
     var expect = join(base, 'expect', prefix);
     glob.sync('**/*', {cwd: expect})

tests/build/expect/build-js-with-deps/a/1.0.0/index-debug.js

@@ -0,0 +1,8 @@
+define("a/1.0.0/index-debug", ["b/1.1.0/src/b-debug", "c/1.1.1/index-debug", "d/0.1.0/index-debug", "d/0.1.1/index-debug"], function(require, exports, module) {
+  require("a/1.0.0/relative-debug");
+  require("d/0.1.1/index-debug");
+});
+define("a/1.0.0/relative-debug", ["b/1.1.0/src/b-debug", "c/1.1.1/index-debug", "d/0.1.0/index-debug"], function(require, exports, module) {
+  console.log('relative');
+  require("b/1.1.0/src/b-debug");
+});

tests/build/expect/build-js-with-deps/b/1.1.0/src/b-debug.js

@@ -0,0 +1,7 @@
+define("b/1.1.0/src/b-debug", ["c/1.1.1/index-debug", "d/0.1.0/index-debug"], function(require, exports, module) {
+  require("c/1.1.1/index-debug");
+  require("b/1.1.0/src/b-debug.tpl");
+});
+define("b/1.1.0/src/b-debug.tpl", [], function(require, exports, module) {
+  module.exports = '<div></div>';
+});

tests/build/expect/build-js-with-deps/c/1.1.1/index-debug.js

@@ -0,0 +1,3 @@
+define("c/1.1.1/index-debug", ["d/0.1.0/index-debug"], function(require, exports, module) {
+  require("d/0.1.0/index-debug");
+});

tests/build/expect/build-js-with-deps/c/1.1.1/index.js

@@ -0,0 +1 @@
+define("c/1.1.1/index",["d/0.1.0/index"],function(d){d("d/0.1.0/index")});

tests/build/expect/build-js-with-deps/d/0.1.0/index-debug.js

@@ -0,0 +1,5 @@
+define("d/0.1.0/index-debug", [], function(require, exports, module) {
+  exports.d = function() {
+    console.log('0.1.0');
+  };
+});

tests/build/expect/build-js-with-deps/d/0.1.0/index.js

@@ -0,0 +1 @@
+define("d/0.1.0/index",[],function(n,o){o.d=function(){console.log("0.1.0")}});

tests/build/expect/build-js-with-deps/d/0.1.1/index-debug.js

@@ -0,0 +1,5 @@
+define("d/0.1.1/index-debug", [], function(require, exports, module) {
+  exports.d = function() {
+    console.log('0.1.1');
+  };
+});

tests/build/expect/build-js-with-deps/d/0.1.1/index.js

@@ -0,0 +1 @@
+define("d/0.1.1/index",[],function(n,o){o.d=function(){console.log("0.1.1")}});