From: Dave Methvin <dave.methvin@gmail.com>
Date: Sun, 6 Jan 2013 17:01:22 +0000 (-0500)
Subject: Fix #12471. Use consistent line endings in jquery.js
X-Git-Tag: 1.9.0rc1~8
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b45c775ca971afab0c915fbc91aa8720a639f5d1;p=jquery.git

Fix #12471. Use consistent line endings in jquery.js
---

diff --git a/Gruntfile.js b/Gruntfile.js
index ea51d180a..0c2c8bb93 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -189,8 +189,8 @@ module.exports = function( grunt ) {
 
 		grunt.verbose.write("Injected sizzle-jquery.js into sizzle.js");
 
-		// Write concatenated source to file
-		grunt.file.write( name, compiled );
+		// Write concatenated source to file, and ensure newline-only termination
+		grunt.file.write( name, compiled.replace( /\x0d\x0a/g, "\x0a" ) );
 
 		// Fail task if errors were logged.
 		if ( this.errorCount ) {
@@ -409,8 +409,16 @@ module.exports = function( grunt ) {
 			nonascii = false;
 
 		distpaths.forEach(function( filename ) {
-			var text = fs.readFileSync( filename, "utf8" ),
-			i, c;
+			var i, c,
+				text = fs.readFileSync( filename, "utf8" );
+
+			// Ensure files use only \n for line endings, not \r\n
+			if ( /\x0d\x0a/.test( text ) ) {
+				grunt.log.writeln( filename + ": Incorrect line endings (\\r\\n)" );
+				nonascii = true;
+			}
+
+			// Ensure only ASCII chars so script tags don't need a charset attribute
 			if ( text.length !== Buffer.byteLength( text, "utf8" ) ) {
 				grunt.log.writeln( filename + ": Non-ASCII characters detected:" );
 				for ( i = 0; i < text.length; i++ ) {
@@ -418,10 +426,10 @@ module.exports = function( grunt ) {
 					if ( c > 127 ) {
 						grunt.log.writeln( "- position " + i + ": " + c );
 						grunt.log.writeln( "-- " + text.substring( i - 20, i + 20 ) );
-						nonascii = true;
 						break;
 					}
 				}
+				nonascii = true;
 			}
 
 			// Modify map so that it points to files in the same folder;