Fix #13983. Switch to //# for sourcemap directives.

This commit is contained in:
Michał Gołębiowski 2013-09-05 18:23:56 +02:00
parent 58a504093c
commit dfaee326e6
3 changed files with 3 additions and 21 deletions

View File

@ -85,12 +85,10 @@ module.exports = function( grunt ) {
"dist/jquery.pre-min.js": [ "dist/jquery.js" ] "dist/jquery.pre-min.js": [ "dist/jquery.js" ]
}, },
options: { 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 %> | " + "/*! jQuery v<%= pkg.version %> | " +
"(c) 2005, 2013 jQuery Foundation, Inc. | " + "(c) 2005, 2013 jQuery Foundation, Inc. | " +
"jquery.org/license\n" + "jquery.org/license */\n"
"//@ sourceMappingURL=jquery.min.map\n" +
"*/\n"
} }
} }
}, },

View File

@ -57,18 +57,6 @@ module.exports = function( grunt ) {
if ( /\.map$/.test( filename ) ) { if ( /\.map$/.test( filename ) ) {
text = text.replace( /"dist\//g, "\"" ); text = text.replace( /"dist\//g, "\"" );
fs.writeFileSync( filename, text, "utf-8" ); 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 // Optionally copy dist files to other locations

View File

@ -38,12 +38,8 @@ module.exports = function( grunt ) {
// Rename the file to a temporary name. // Rename the file to a temporary name.
fs.renameSync( mapFileName, mapping.dest); fs.renameSync( mapFileName, mapping.dest);
grunt.file.write( mapFileName, grunt.file.read( 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. // Refer to the source jquery.js, not the temporary jquery.pre-min.js.
.replace( /\.pre-min\./g, "." ) .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, "" ));
}); });
// Remove temporary files. // Remove temporary files.