Build: Remove empty define({}) from build output

dcorb · dmethvin · commit 2138f158be66 · 2014-12-04T16:48:28.000-05:00
Fixes gh-1768 Closes gh-1569 (cherry picked from commit 2c1b556)
diff --git a/build/tasks/build.js b/build/tasks/build.js
@@ -20,6 +20,8 @@ module.exports = function( grunt ) {
 			optimize: "none",
 			// Include dependencies loaded with require
 			findNestedDependencies: true,
+			// Avoid inserting define() placeholder
+			skipModuleInsertion: true,
 			// Avoid breaking semicolons inserted by r.js
 			skipSemiColonInsertion: true,
 			wrap: {
@@ -81,7 +83,7 @@ module.exports = function( grunt ) {
 
 			// Remove empty definitions
 			contents = contents
-				.replace( /define\(\[[^\]]+\]\)[\W\n]+$/, "" );
+				.replace( /define\(\[[^\]]*\]\)[\W\n]+$/, "" );
 		}
 		// AMD Name
 		if ( (amdName = grunt.option( "amd" )) != null && /^exports\/amd$/.test( name ) ) {
