prevented composite tasks from breaking if dependent tasks are not selected, closes #16

Author: Sebastian Henneberg

index.js

@@ -36,15 +36,15 @@ module.exports = function(gulp, tasks, config) {
   }
 
   if (tasks.indexOf('index') !== -1) {
-    require('./tasks/index')(gulp, config.bases.dist, config.app.index, config.appName);
+    require('./tasks/index')(gulp, tasks, config.bases.dist, config.app.index, config.appName);
   }
 
   if (tasks.indexOf('jshint') !== -1) {
     require('./tasks/jshint')(gulp, config.app.alljs);
   }
 
   if (tasks.indexOf('karma') !== -1) {
-    require('./tasks/karma')(gulp, config.dirname);
+    require('./tasks/karma')(gulp, tasks,  config.dirname);
   }
 
   if (tasks.indexOf('mavenDeploy') !== -1) {
@@ -72,7 +72,7 @@ module.exports = function(gulp, tasks, config) {
   }
 
   if (tasks.indexOf('sass') !== -1) {
-    require('./tasks/sass')(gulp, config.app.scss, config.dist.css, config.env.rev, config.dist.fonts, config.sourceMapsPath);
+    require('./tasks/sass')(gulp, tasks, config.app.scss, config.dist.css, config.env.rev, config.dist.fonts, config.sourceMapsPath);
   }
 
   if (tasks.indexOf('scripts') !== -1) {
@@ -84,19 +84,19 @@ module.exports = function(gulp, tasks, config) {
   }
 
   if (tasks.indexOf('vendorFonts') !== -1) {
-    require('./tasks/vendorFonts')(gulp, config.debug, config.dist.fonts, config.env.rev);
+    require('./tasks/vendorFonts')(gulp, tasks, config.debug, config.dist.fonts, config.env.rev);
   }
 
   if (tasks.indexOf('vendorScripts') !== -1) {
-    require('./tasks/vendorScripts')(gulp, config.dist.js, config.sourceMapsPath, config.debug, config.env.rev);
+    require('./tasks/vendorScripts')(gulp, tasks, config.dist.js, config.sourceMapsPath, config.debug, config.env.rev);
   }
 
   if (tasks.indexOf('vendorStyles') !== -1) {
-    require('./tasks/vendorStyles')(gulp, config.dist.css, config.sourceMapsPath, config.debug, config.env.rev);
+    require('./tasks/vendorStyles')(gulp, tasks, config.dist.css, config.sourceMapsPath, config.debug, config.env.rev);
   }
 
   if (tasks.indexOf('watch') !== -1) {
-    require('./tasks/watch')(gulp, config);
+    require('./tasks/watch')(gulp, tasks, config);
   }
 
 };

package.json

@@ -67,6 +67,7 @@
     "jshint-stylish": "^2.0.1",
     "karma": "^0.12.36",
     "main-bower-files": "^2.8.0",
-    "run-sequence": "^1.1.1"
+    "run-sequence": "^1.1.1",
+    "underscore": "^1.8.3"
   }
 }

tasks/index.js

@@ -4,9 +4,10 @@ var extend = require('extend'),
   gulpInject = require('gulp-inject'),
   minifyInline = require('gulp-minify-inline'),
   minifyHtml = require('gulp-minify-html'),
-  preprocess = require('gulp-preprocess');
+  preprocess = require('gulp-preprocess'),
+  _ = require('underscore');
 
-module.exports = function(gulp, dest, index, appName) {
+module.exports = function(gulp, tasks, dest, index, appName) {
   function injectIndex() {
     // don't read, just insert paths
     var srcOptions = {
@@ -44,6 +45,6 @@ module.exports = function(gulp, dest, index, appName) {
   gulp.task('justIndex', injectIndex);
 
   // use this initial building
-  gulp.task('index', ['partials', 'configScripts', 'scripts', 'vendorScripts', 'styles', 'vendorStyles'], injectIndex);
+  var mergedTasks = _.intersection(tasks, ['partials', 'configScripts', 'scripts', 'vendorScripts', 'styles', 'vendorStyles']);
+  gulp.task('index', mergedTasks, injectIndex);
 };
-

tasks/karma.js

@@ -1,10 +1,11 @@
 'use strict';
 
 var karma = require('karma').server,
-  runSequence = require('run-sequence');
+  runSequence = require('run-sequence'),
+  _ = require('underscore');
 
 
-module.exports = function(gulp, config) {
+module.exports = function(gulp, tasks, config) {
   var configFile = config + '/karma.conf.js';
   var browsers = ['PhantomJS', 'Firefox', 'Chrome'];
 
@@ -37,17 +38,17 @@ module.exports = function(gulp, config) {
     });
   });
 
+  var mergedTasks = _.intersection(tasks, ['config', 'partials', 'vendorScripts']);
+
   gulp.task('test', function() {
-    runSequence(['config', 'partials', 'vendorScripts'], 'karma');
+    runSequence(mergedTasks, 'karma');
   });
 
   gulp.task('test:all', function() {
-    runSequence(['config', 'partials', 'vendorScripts'], 'karma:all');
+    runSequence(mergedTasks, 'karma:all');
   });
 
   gulp.task('test:watch', function() {
-    runSequence(['config', 'partials', 'vendorScripts'], 'karma:watch');
+    runSequence(mergedTasks, 'karma:watch');
   });
 };
-
-

tasks/sass.js

@@ -3,11 +3,13 @@ var sass = require('gulp-sass'),
   autoPrefixer = require('gulp-autoprefixer'),
   gulpif = require('gulp-if'),
   rev = require('gulp-rev'),
-  revReplace = require('gulp-rev-replace');
+  revReplace = require('gulp-rev-replace'),
+  _ = require('underscore');
 
 
-module.exports = function(gulp, src, dest, revFlag, manifestPath, sourceMapsPath) {
-  gulp.task('styles', ['fonts'], function (done) {
+module.exports = function(gulp, tasks, src, dest, revFlag, manifestPath, sourceMapsPath) {
+  var mergedTasks = _.intersection(tasks, ['fonts']);
+  gulp.task('styles', mergedTasks, function (done) {
     var optionsSass = {
       outputStyle: 'compressed'
     };
@@ -32,4 +34,3 @@ module.exports = function(gulp, src, dest, revFlag, manifestPath, sourceMapsPath
     done();
   });
 };
-

tasks/vendorFonts.js

@@ -3,14 +3,16 @@
 var bowerFiles = require('main-bower-files'),
   gulpif = require('gulp-if'),
   rev = require('gulp-rev'),
-  debug = require('gulp-debug');
+  debug = require('gulp-debug'),
+  _ = require('underscore');
 
 var fontsFilter = {
   filter: /\.(otf|eot|svg|ttf|woff)/i
 };
 
-module.exports = function(gulp, debugFlag, dest, revFlag) {
-  gulp.task('vendorFonts', ['bower:install', 'bower:prune'], function() {
+module.exports = function(gulp, tasks, debugFlag, dest, revFlag) {
+  var mergedTasks = _.intersection(tasks, ['bower:install', 'bower:prune']);
+  gulp.task('vendorFonts', mergedTasks, function() {
     return gulp.src(bowerFiles(fontsFilter))
       .pipe(gulpif(debugFlag, debug()))
       .pipe(gulpif(revFlag, rev()))

tasks/vendorScripts.js

@@ -7,7 +7,8 @@ var bowerFiles = require('main-bower-files'),
   newer = require('gulp-newer'),
   concat = require('gulp-concat'),
   rev = require('gulp-rev'),
-  uglify = require('gulp-uglify');
+  uglify = require('gulp-uglify'),
+  _ = require('underscore');
 
 
 var isFirstRun = true;
@@ -16,8 +17,9 @@ var jsFilter = {
   filter: /\.js$/i
 };
 
-module.exports = function(gulp, dest, sourceMapPath, debugFlag, revFlag) {
-  gulp.task('vendorScripts', ['bower:install', 'bower:prune'], function() {
+module.exports = function(gulp, tasks, dest, sourceMapPath, debugFlag, revFlag) {
+  var mergedTasks = _.intersection(tasks, ['bower:install', 'bower:prune']);
+  gulp.task('vendorScripts', mergedTasks, function() {
     var result = gulp.src(bowerFiles(jsFilter))
       .pipe(gulpif(debugFlag, debug()))
       .pipe(sourceMaps.init())

tasks/vendorStyles.js

@@ -7,7 +7,8 @@ var bowerFiles = require('main-bower-files'),
   newer = require('gulp-newer'),
   concat = require('gulp-concat'),
   rev = require('gulp-rev'),
-  minifyCSS = require('gulp-minify-css');
+  minifyCSS = require('gulp-minify-css'),
+  _ = require('underscore');
 
 
 var isFirstRun = true;
@@ -16,8 +17,9 @@ var cssFilter = {
   filter: /\.css$/i
 };
 
-module.exports = function(gulp, dest, sourceMapsPath, debugFlag, revFlag) {
-  gulp.task('vendorStyles', ['bower:install', 'bower:prune'], function(done) {
+module.exports = function(gulp, tasks, dest, sourceMapsPath, debugFlag, revFlag) {
+  var mergedTasks = _.intersection(tasks, ['bower:install', 'bower:prune']);
+  gulp.task('vendorStyles', mergedTasks, function(done) {
     gulp.src(bowerFiles(cssFilter))
       .pipe(gulpif(debugFlag, debug()))
       .pipe(sourceMaps.init())

tasks/watch.js

@@ -2,13 +2,23 @@
 
 var liveReload = require('gulp-livereload');
 
-module.exports = function(gulp, config) {
+module.exports = function(gulp, tasks, config) {
   gulp.task('watch', ['build'], function() {
     liveReload.listen();
     gulp.watch(config.app.index, ['justIndex']);
-    gulp.watch(config.app.views, ['partials']);
-    gulp.watch(config.app.statics, ['statics']);
-    gulp.watch(config.app.images, ['images']);
+
+    if (tasks.indexOf('partials')) {
+      gulp.watch(config.app.views, ['partials']);
+    }
+
+    if (tasks.indexOf('partials')) {
+      gulp.watch(config.app.statics, ['statics']);
+    }
+
+    if (tasks.indexOf('partials')) {
+      gulp.watch(config.app.images, ['images']);
+    }
+
     gulp.watch(config.app.scssAll, ['styles']);
     gulp.watch(config.app.js, ['scripts']);
     gulp.watch(config.bowerjson, ['vendorScripts', 'vendorStyles', 'vendorFonts']);