From dfaee326e6605dc6d848c4c5022e39069d2cee65 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82e=CC=A8biowski?= Date: Thu, 5 Sep 2013 18:23:56 +0200 Subject: [PATCH] Fix #13983. Switch to //# for sourcemap directives. --- Gruntfile.js | 6 ++---- build/tasks/dist.js | 12 ------------ build/tasks/uglify.js | 6 +----- 3 files changed, 3 insertions(+), 21 deletions(-) diff --git a/Gruntfile.js b/Gruntfile.js index 6f75d3925..c8060dc24 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -85,12 +85,10 @@ module.exports = function( grunt ) { "dist/jquery.pre-min.js": [ "dist/jquery.js" ] }, options: { - banner: "\n\n\n\n\n\n\n\n\n\n" + // banner line size must be preserved + banner: "\n\n\n\n\n\n\n\n\n\n\n\n" + // banner line size must be preserved "/*! jQuery v<%= pkg.version %> | " + "(c) 2005, 2013 jQuery Foundation, Inc. | " + - "jquery.org/license\n" + - "//@ sourceMappingURL=jquery.min.map\n" + - "*/\n" + "jquery.org/license */\n" } } }, diff --git a/build/tasks/dist.js b/build/tasks/dist.js index 591078cde..97d882e70 100644 --- a/build/tasks/dist.js +++ b/build/tasks/dist.js @@ -57,18 +57,6 @@ module.exports = function( grunt ) { if ( /\.map$/.test( filename ) ) { text = text.replace( /"dist\//g, "\"" ); fs.writeFileSync( filename, text, "utf-8" ); - - // Use our hard-coded sourceMap directive instead of the autogenerated one (#13274; #13776) - } else if ( /\.min\.js$/.test( filename ) ) { - i = 0; - text = text.replace( /(?:\/\*|)\n?\/\/@\s*sourceMappingURL=.*(\n\*\/|)/g, - function( match ) { - if ( i++ ) { - return ""; - } - return match; - }); - fs.writeFileSync( filename, text, "utf-8" ); } // Optionally copy dist files to other locations diff --git a/build/tasks/uglify.js b/build/tasks/uglify.js index 7c1a01bec..ccbc70888 100644 --- a/build/tasks/uglify.js +++ b/build/tasks/uglify.js @@ -38,12 +38,8 @@ module.exports = function( grunt ) { // Rename the file to a temporary name. fs.renameSync( mapFileName, mapping.dest); grunt.file.write( mapFileName, grunt.file.read( mapping.dest ) - // The uglify task erroneously prepends dist/ to file names. - .replace( /"dist\//g, "\"" ) // Refer to the source jquery.js, not the temporary jquery.pre-min.js. - .replace( /\.pre-min\./g, "." ) - // There's already a pragma at the beginning of the file, remove the one at the end. - .replace( /\/\/@ sourceMappingURL=jquery\.min\.map$/g, "" )); + .replace( /\.pre-min\./g, "." )); }); // Remove temporary files. -- 2.39.5