Merge pull request #177 from juliandescottes/enhance-build

Enhance build

.gitignore

@@ -12,15 +12,12 @@ npm-debug.log
 # git stackdumps
 *.stackdump
 
-# builds
-build/*.js
-build/*.css
 
 # diffs
 diff.txt
 
-# Closure compiler generated JS binary.
-build/closure/closure_compiled_binary.js
+# build destination
+dest
 
 # marked as private
 *.private.*

Gruntfile.js

@@ -36,6 +36,10 @@ module.exports = function(grunt) {
   };
 
   grunt.initConfig({
+    clean: {
+      before: ['dest'],
+      after: ['build/closure/closure_compiled_binary.js']
+    },
     jshint: {
       options: {
         indent:2,
@@ -60,12 +64,29 @@ module.exports = function(grunt) {
           base : '.',
           port : 4321
         }
-      },
-      serve : {
-        options : {
-          base : '.',
-          port : 1234,
-          keepalive : true
+      }
+    },
+    express: {
+      server: {
+        options: {
+          port: 9001,
+          hostname : 'localhost',
+          bases: ['dest']
+        }
+      }
+    },
+    open : {
+      server : {
+        path : 'http://localhost:9001/'
+      }
+    },
+
+    watch: {
+      scripts: {
+        files: ['src/**/*.*'],
+        tasks: ['merge'],
+        options: {
+          spawn: false
         }
       }
     },
@@ -79,11 +100,11 @@ module.exports = function(grunt) {
           separator : ';'
         },
         src : piskelScripts,
-        dest : 'build/piskel-packaged.js'
+        dest : 'dest/js/piskel-packaged.js'
       },
       css : {
         src : piskelStyles,
-        dest : 'build/piskel-style-packaged.css'
+        dest : 'dest/css/piskel-style-packaged.css'
       }
     },
     uglify : {
@@ -92,10 +113,20 @@ module.exports = function(grunt) {
       },
       my_target : {
         files : {
-          'build/piskel-packaged-min.js' : ['build/piskel-packaged.js']
+          'dest/js/piskel-packaged-min.js' : ['dest/js/piskel-packaged.js']
         }
       }
     },
+    copy: {
+      main: {
+        files: [
+          {src: ['src/piskel-boot.js'], dest: 'dest/piskel-boot.js'},
+          {src: ['src/js/lib/iframeLoader.js'], dest: 'dest/js/lib/iframeLoader.js'},
+          {expand: true, src: ['img/**'], cwd: 'src/', dest: 'dest/', filter: 'isFile'},
+          {expand: true, src: ['**/*.html'], cwd: 'src/', dest: 'dest/', filter: 'isFile'}
+        ]
+      }
+    },
     closureCompiler:  {
       options: {
         // [REQUIRED] Path to closure compiler
@@ -161,31 +192,36 @@ module.exports = function(grunt) {
     src: ['src/css/**/*.css']
   });
 
-  grunt.loadNpmTasks('grunt-contrib-connect');
-  grunt.loadNpmTasks('grunt-contrib-jshint');
-  grunt.loadNpmTasks('grunt-contrib-concat');
-  grunt.loadNpmTasks('grunt-contrib-uglify');
   grunt.loadNpmTasks('grunt-closure-tools');
+  grunt.loadNpmTasks('grunt-contrib-clean');
+  grunt.loadNpmTasks('grunt-contrib-concat');
+  grunt.loadNpmTasks('grunt-contrib-connect');
+  grunt.loadNpmTasks('grunt-contrib-copy');
+  grunt.loadNpmTasks('grunt-contrib-jshint');
+  grunt.loadNpmTasks('grunt-contrib-uglify');
+  grunt.loadNpmTasks('grunt-contrib-watch');
+  grunt.loadNpmTasks('grunt-express');
   grunt.loadNpmTasks('grunt-ghost');
+  grunt.loadNpmTasks('grunt-open');
   grunt.loadNpmTasks('grunt-leading-indent');
 
   // Validate
   grunt.registerTask('lint', ['leadingIndent:jsFiles', 'leadingIndent:cssFiles', 'jshint']);
 
   // Validate & Test
-  grunt.registerTask('test', ['leadingIndent:jsFiles', 'leadingIndent:cssFiles', 'jshint', 'compile', 'connect:test', 'ghost:default']);
+  grunt.registerTask('test', ['lint', 'compile', 'connect:test', 'ghost:default']);
 
   // Validate & Test (faster version) will NOT work on travis !!
-  grunt.registerTask('precommit', ['leadingIndent:jsFiles', 'leadingIndent:cssFiles', 'jshint', 'compile', 'connect:test', 'ghost:local']);
+  grunt.registerTask('precommit', ['lint', 'compile', 'connect:test', 'ghost:local']);
 
   // Compile JS code (eg verify JSDoc annotation and types, no actual minified code generated).
-  grunt.registerTask('compile', ['closureCompiler:compile']);
+  grunt.registerTask('compile', ['closureCompiler:compile', 'clean:after']);
 
-  grunt.registerTask('merge',  ['concat:js', 'concat:css', 'uglify']);
+  grunt.registerTask('merge',  ['concat:js', 'concat:css', 'uglify', 'copy']);
 
   // Validate & Build
-  grunt.registerTask('default', ['leadingIndent:jsFiles', 'leadingIndent:cssFiles', 'jshint', 'concat:js', 'concat:css', 'compile', 'uglify']);
+  grunt.registerTask('default', ['clean:before', 'lint', 'compile', 'merge']);
 
-  // Start webserver
-  grunt.registerTask('serve', ['connect:serve']);
+  // Start webserver and watch for changes
+  grunt.registerTask('server', ['express', 'open', 'watch']);
 };

package.json

@@ -13,11 +13,16 @@
   },
   "devDependencies": {
     "grunt": "~0.4.1",
+    "grunt-contrib-clean": "0.5.0",
     "grunt-contrib-connect": "0.3.0",
     "grunt-contrib-concat": "0.1.2",
+    "grunt-contrib-copy": "0.5.0",
     "grunt-contrib-jshint": "0.5.4",
     "grunt-contrib-uglify": "0.2.2",
+    "grunt-contrib-watch": "0.6.1",
+    "grunt-express": "1.0",
     "grunt-ghost": "1.0.12",
+    "grunt-open": "0.2.3",
     "grunt-leading-indent": "0.1.0",
     "grunt-closure-tools": "~0.8.3"
   }

src/piskel-boot.js

@@ -59,11 +59,11 @@
   } else {
     var script;
     if (window.location.href.indexOf("pack") != -1) {
-      script = "build/piskel-packaged.js";
+      script = "js/piskel-packaged.js";
     } else {
-      script = "build/piskel-packaged-min.js";
+      script = "js/piskel-packaged-min.js";
     }
-    loadStyle('build/piskel-style-packaged.css');
+    loadStyle('css/piskel-style-packaged.css');
 
     var loaderInterval = window.setInterval(function () {
       if (document.querySelectorAll("[data-iframe-loader]").length === 0) {