Commit d53ddc90 authored by Dave Methvin's avatar Dave Methvin

Fix #13983. Switch to //# for sourcemaps.

Normally we'd avoid doing this on a patch release, but Chrome Canary is
spewing deprecation warnings already and that's generating bug tickets.
parent 5f1aa235
...@@ -115,7 +115,7 @@ module.exports = function( grunt ) { ...@@ -115,7 +115,7 @@ module.exports = function( grunt ) {
"/*! 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" + "//# sourceMappingURL=jquery.min.map\n" +
"*/\n" "*/\n"
} }
} }
......
...@@ -167,10 +167,10 @@ function makeReleaseCopies( next ) { ...@@ -167,10 +167,10 @@ function makeReleaseCopies( next ) {
} else if ( /\.min\.js$/.test( releaseFile ) ) { } else if ( /\.min\.js$/.test( releaseFile ) ) {
// Minified files point back to the corresponding map; // Minified files point back to the corresponding map;
// again assume one big happy directory. // again assume one big happy directory.
// "//@ sourceMappingURL=jquery.min.map" // "//# sourceMappingURL=jquery.min.map"
text = fs.readFileSync( builtFile, "utf8" ) text = fs.readFileSync( builtFile, "utf8" )
.replace( /\/\/@ sourceMappingURL=\S+/, .replace( /\/\/# sourceMappingURL=\S+/,
"//@ sourceMappingURL=" + unpathedFile.replace( /\.js$/, ".map" ) ); "//# sourceMappingURL=" + unpathedFile.replace( /\.js$/, ".map" ) );
fs.writeFileSync( releaseFile, text ); fs.writeFileSync( releaseFile, text );
} else if ( builtFile !== releaseFile ) { } else if ( builtFile !== releaseFile ) {
copy( builtFile, releaseFile ); copy( builtFile, releaseFile );
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment