move some stuff for reusability between modularized tasks

build-system/internal-version.js

@@ -0,0 +1,19 @@
+/**
+ * Copyright 2015 The AMP HTML Authors. All Rights Reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS-IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+// Used to e.g. references the ads binary from the runtime to get
+// version lock.
+exports.VERSION = new Date().getTime();

build-system/tasks/clean.js

@@ -0,0 +1,31 @@
+/**
+ * Copyright 2015 The AMP HTML Authors. All Rights Reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS-IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+var del = require('del');
+var gulp = require('gulp-help')(require('gulp'));
+
+
+/**
+ * Clean up the build artifacts
+ *
+ * @param {function} done callback
+ */
+function clean(done) {
+  del(['dist', 'dist.3p', 'build', 'examples.build'], done);
+}
+
+
+gulp.task('clean', 'Removes build output', clean);

build-system/tasks/index.js

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+require('./clean');
 require('./lint');
 require('./make-golden');
 require('./presubmit-checks');

gulpfile.js

@@ -19,7 +19,6 @@ var babel = require('babelify');
 var browserify = require('browserify');
 var buffer = require('vinyl-buffer');
 var cssnano = require('cssnano');
-var del = require('del');
 var file = require('gulp-file');
 var fs = require('fs-extra');
 var gulp = require('gulp-help')(require('gulp'));
@@ -34,6 +33,7 @@ var uglify = require('gulp-uglify');
 var util = require('gulp-util');
 var watchify = require('watchify');
 var wrap = require('gulp-wrap');
+var internalRuntimeVersion = require('./build-system/internal-version').VERSION;
 require('./build-system/tasks');
 
 // NOTE: see https://github.com/ai/browserslist#queries for `browsers` list
@@ -54,9 +54,6 @@ cssnano = cssnano({
   zindex: false
 });
 
-// Used to e.g. references the ads binary from the runtime to get
-// version lock.
-var internalRuntimeVersion = new Date().getTime();
 
 /**
  * Build all the AMP extensions
@@ -85,14 +82,6 @@ function buildExtensions(options) {
   buildExtension('amp-youtube', '0.1', false, options);
 }
 
-/**
- * Clean up the build artifacts
- *
- * @param {function} done callback
- */
-function clean(done) {
-  del(['dist', 'dist.3p', 'build', 'examples.build'], done);
-}
 
 /**
  * Compile the polyfills script and drop it in the build folder
@@ -452,7 +441,6 @@ function compileJs(srcDir, srcFilename, destDir, options) {
  * Gulp tasks
  */
 gulp.task('build', 'Builds the AMP library', build);
-gulp.task('clean', 'Removes build output', clean);
 gulp.task('css', 'Recompile css to build directory', compileCss);
 gulp.task('default', 'Same as "watch"', ['watch']);
 gulp.task('dist', 'Build production binaries', dist);