Swapped out adm-zip with node-zip since it was also broken

jbemmz · Apr 12, 2013 · d4087d5 · d4087d5
1 parent 251bac2
commit d4087d5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -12,7 +12,7 @@
 		"jshint": ">= 0.9.1",
 		"uglify-js": ">= 2.0.0",
 		"glob": ">= 3.1.12",
-		"adm-zip": ">= 0.4.3",
+		"node-zip": ">= 1.0.1",
 		"less": ">= 1.3.1"
 	}
 }
diff --git a/tools/BuildTools.js b/tools/BuildTools.js
@@ -272,7 +272,7 @@ exports.jshint = function (options) {
 };
 
 exports.zip = function (options) {
-	var AdmZip = require('adm-zip');
+	var AdmZip = require('node-zip');
 	var archive = new AdmZip();
 
 	function process(filePath, zipFilePath) {
@@ -305,15 +305,15 @@ exports.zip = function (options) {
 		}
 
 		if (stat.isFile()) {
-			var data = fs.readFileSync(filePath);
+			var data = fs.readFileSync(filePath, 'binary');
 
 			if (options.dataFilter) {
 				args = {filePath: filePath, zipFilePath: zipFilePath, data: data};
 				options.dataFilter(args);
 				data = args.data;
 			}
 
-			archive.addFile(path.join(options.baseDir, zipFilePath), data);
+			archive.file(path.join(options.baseDir, zipFilePath), data);
 		} else if (stat.isDirectory()) {
 			fs.readdirSync(filePath).forEach(function(fileName) {
 				process(path.join(filePath, fileName), path.join(zipFilePath, fileName));
@@ -331,7 +331,7 @@ exports.zip = function (options) {
 		}
 	});
 
-	archive.writeZip(options.to);
+	fs.writeFileSync(options.to, archive.generate({base64: false, compression: 'DEFLATE'}), 'binary');
 };
 
 exports.compileAmd = function (options) {