Merge pull request #856 from spmjs/option-skip

add skip

Author: sorrycc

bin/spm-build

@@ -17,6 +17,7 @@ commander
   .option('-f, --force', 'force option')
   .option('--include [include]', 'determine which files will be included, optional: self, relative, all, standalone')
   .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('--zip', 'archive by zip')
   .parse(process.argv);
@@ -41,6 +42,7 @@ var args = {
   cwd: commander.inputDirectory,
   include: commander.include,
   ignore: commander.ignore,
+  skip: commander.skip,
   idleading: commander.idleading,
   zip: commander.zip,
   force: commander.force,

lib/build.js

@@ -40,7 +40,8 @@ function build(args, cb) {
   // add package info
   try {
     var pkg = new Package(args.cwd, {
-      extraDeps: {handlebars: 'handlebars-runtime'}
+      extraDeps: {handlebars: 'handlebars-runtime'},
+      skip: args.skip
     });
     log.info('package', 'analyse infomation');
     log.info('package', 'dependencies: ' + Object.keys(pkg.dependencies));

package.json

@@ -55,7 +55,7 @@
     "serve-spm": "^0.3.1",
     "spmrc": "~1.0.0",
     "tar": "~0.1.19",
-    "umi": "~0.3.0",
+    "umi": "~0.4.0",
     "vinyl-source-stream": "~0.1.0",
     "win-spawn": "~2.0.0"
   },

tests/build.js

@@ -184,6 +184,19 @@ describe('build', function() {
     });
   });
 
+  it('can ignore when parse', function(done) {
+    var opt = {
+      cwd: join(base, 'ignore'),
+      dest: dest,
+      skip: ['crypto']
+    };
+    build(opt, function(err) {
+      should.not.exist(err);
+      assets('ignore', dest);
+      done();
+    });
+  });
+
   function assets(prefix, dest) {
     var expect = join(base, 'expect', prefix);
     glob.sync('**/*', {cwd: expect})

tests/build/expect/ignore/a/1.0.0/index-debug.js

@@ -0,0 +1,13 @@
+define("a/1.0.0/index-debug", [], function(require, exports, module) {
+  (function() {
+    var _rng;
+    if (typeof require === 'function') {
+      try {
+        var _rb = require("crypto").randomBytes;
+        _rng = _rb && function() {
+          return _rb(16);
+        };
+      } catch (e) {}
+    }
+  })();
+});

tests/build/expect/ignore/a/1.0.0/index.js

@@ -0,0 +1 @@
+define("a/1.0.0/index",[],function(n){!function(){var t;if("function"==typeof n)try{var c=n("crypto").randomBytes;t=c&&function(){return c(16)}}catch(f){}}()});

tests/build/ignore/index.js

@@ -0,0 +1,11 @@
+(function() {
+  var _rng;
+  if (typeof require === 'function') {
+    try {
+      var _rb = require('crypto').randomBytes;
+      _rng = _rb && function() {
+        return _rb(16);
+      };
+    } catch (e) {}
+  }
+})();

tests/build/ignore/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "a",
+  "version": "1.0.0"
+}