From 11f386fe89b7b6b86e6c87e5bb22cc34535138b9 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82e=CC=A8biowski?= Date: Wed, 2 Oct 2013 00:39:49 +0200 Subject: [PATCH] No ticket. Simplify the post-uglify task. (cherry-picked from 923bfc80dbba00659a2fbadafad6d894973cc560) --- Gruntfile.js | 7 ++----- build/tasks/uglify.js | 14 +++++--------- 2 files changed, 7 insertions(+), 14 deletions(-) diff --git a/Gruntfile.js b/Gruntfile.js index a4215b558..4be86f296 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -121,12 +121,9 @@ module.exports = function( grunt ) { }, "post-uglify": { all: { - files: { - "dist/jquery.min.map.tmp": [ "dist/jquery.min.map" ], - "dist/jquery.min.js.tmp": [ "dist/jquery.min.js" ] - }, + src: [ "dist/jquery.min.map" ], options: { - tempFiles: [ "dist/jquery.min.map.tmp", "dist/jquery.min.js.tmp", "dist/jquery.pre-min.js" ] + tempFiles: [ "dist/jquery.pre-min.js" ] } } } diff --git a/build/tasks/uglify.js b/build/tasks/uglify.js index ccbc70888..f8df5e77e 100644 --- a/build/tasks/uglify.js +++ b/build/tasks/uglify.js @@ -18,10 +18,10 @@ module.exports = function( grunt ) { // Remove the main jQuery banner, it'll be replaced by the new banner anyway. .replace( /^\/\*![\W\w]*?\*\/\n?/g, "" ) // Strip other banners preserving line count. - .replace( /^\/\*!(?:.|\n)*?\*\/\n?/gm, function ( match ) { + .replace( /^\/\*!(?:.|\n)*?\*\/\n?/gm, function( match ) { return match.replace( /[^\n]/gm, "" ); }); - }).join("\n"); + }).join( "\n" ); // Write temp file (with optional banner) grunt.file.write( mapping.dest, ( banner || "" ) + input ); @@ -30,16 +30,12 @@ module.exports = function( grunt ) { // Change the map file to point back to jquery.js instead of jquery.pre-min.js. // The problem is caused by the pre-uglify task. - // Also, remove temporary files. grunt.registerMultiTask( "post-uglify", function() { this.files.forEach(function( mapping ) { - var mapFileName = mapping.src[ 0 ]; - - // Rename the file to a temporary name. - fs.renameSync( mapFileName, mapping.dest); - grunt.file.write( mapFileName, grunt.file.read( mapping.dest ) + mapping.src.forEach( function( src ) { // Refer to the source jquery.js, not the temporary jquery.pre-min.js. - .replace( /\.pre-min\./g, "." )); + grunt.file.write( src, grunt.file.read( src ).replace( /\.pre-min\./g, "." ) ); + }); }); // Remove temporary files. -- 2.39.5