Skip to content
Permalink
Browse files

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.
  • Loading branch information
dmethvin committed Jul 2, 2013
1 parent 5f1aa23 commit d53ddc90c1f119fb9148a553443ef3fbc3f3cc99
Showing with 4 additions and 4 deletions.
  1. +1 −1 Gruntfile.js
  2. +3 −3 build/release.js
@@ -115,7 +115,7 @@ module.exports = function( grunt ) {
"/*! jQuery v<%= pkg.version %> | " +
"(c) 2005, 2013 jQuery Foundation, Inc. | " +
"jquery.org/license\n" +
"//@ sourceMappingURL=jquery.min.map\n" +
"//# sourceMappingURL=jquery.min.map\n" +
"*/\n"
}
}
@@ -167,10 +167,10 @@ function makeReleaseCopies( next ) {
} else if ( /\.min\.js$/.test( releaseFile ) ) {
// Minified files point back to the corresponding map;
// again assume one big happy directory.
// "//@ sourceMappingURL=jquery.min.map"
// "//# sourceMappingURL=jquery.min.map"
text = fs.readFileSync( builtFile, "utf8" )
.replace( /\/\/@ sourceMappingURL=\S+/,
"//@ sourceMappingURL=" + unpathedFile.replace( /\.js$/, ".map" ) );
.replace( /\/\/# sourceMappingURL=\S+/,
"//# sourceMappingURL=" + unpathedFile.replace( /\.js$/, ".map" ) );
fs.writeFileSync( releaseFile, text );
} else if ( builtFile !== releaseFile ) {
copy( builtFile, releaseFile );

0 comments on commit d53ddc9

Please sign in to comment.
You can’t perform that action at this time.