From: Jörn Zaefferer <joern.zaefferer@gmail.com>
Date: Mon, 19 Mar 2012 22:08:56 +0000 (+0100)
Subject: Build/grunt: Another attempt at using adm-zip. Keep the result in place, but don... 
X-Git-Tag: 1.9.0m8~184
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2a1ee3725818c339ab45894db0347e016188e2d8;p=jquery-ui.git

Build/grunt: Another attempt at using adm-zip. Keep the result in place, but don't (yet) use it - need to report issues to adm-zip
---

diff --git a/grunt.js b/grunt.js
index e7395285a..83522ebb4 100644
--- a/grunt.js
+++ b/grunt.js
@@ -308,24 +308,24 @@ task.registerMultiTask( "copy", "Copy files to destination folder and replace @V
 
 
 task.registerMultiTask( "zip", "Create a zip file for release", function() {
-	var done = this.async();
 	// TODO switch back to adm-zip for better cross-platform compability once it actually works
-	// 0.1.2 doesn't compress properly (or at all)
-
-	// var files = file.expand(this.file.src);
-	// log.writeln("Creating zip file " + this.file.dest);
+	// 0.1.3 works, but result can't be unzipped
+	// its also a lot slower then zip program, probably due to how its used...
+	// var files = file.expand( "dist/" + this.file.src + "/**/*" );
+	// log.writeln( "Creating zip file " + this.file.dest );
 
-	// var fs = require('fs');
-	// var AdmZip = require('adm-zip');
+	// var fs = require( "fs" );
+	// var AdmZip = require( "adm-zip" );
 	// var zip = new AdmZip();
-	// files.forEach(function(file) {
-	//   log.verbose.writeln('Zipping ' + file);
-	//   // rewrite file names from dist folder (created by build), drop the /dist part
-	//   zip.addFile(file.replace(/^dist/, ''), fs.readFileSync(file));
+	// files.forEach(function( file ) {
+	// 	log.verbose.writeln( "Zipping " + file );
+	// 	// rewrite file names from dist folder (created by build), drop the /dist part
+	// 	zip.addFile(file.replace(/^dist/, "" ), fs.readFileSync( file ) );
 	// });
-	// zip.writeZip(this.file.dest);
-	// log.writeln("Wrote " + files.length + " files to " + this.file.dest);
+	// zip.writeZip( "dist/" + this.file.dest );
+	// log.writeln( "Wrote " + files.length + " files to " + this.file.dest );
 
+	var done = this.async();
 	var dest = this.file.dest;
 	var src = template.process( this.file.src, config() );
 	utils.spawn({
diff --git a/package.json b/package.json
index cd0c9f78d..02d7a1173 100644
--- a/package.json
+++ b/package.json
@@ -29,7 +29,8 @@
     "grunt": "0.2.x",
     "sqwish": "0.2.x",
     "request": "2.9.x",
-    "csslint": "0.9.x"
+    "csslint": "0.9.x",
+    "adm-zip": "0.1.x"
   },
   "keywords": []
 }
\ No newline at end of file