From 9ec09c3b4aa5182c2a8b8f51afb861b685a4003c Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 13 May 2019 21:55:45 +0200 Subject: [PATCH] Build: Fix the regex parsing AMD var-modules (#4389) The previous regex caused the final jQuery binary to have syntax errors for var-modules with names starting with "return". For example, the following module wouldn't work when the file is named `returnTrue.js`: ```js define( function() { "use strict"; return function returnTrue() { return true; }; } ); ``` Closes gh-4389 --- build/tasks/build.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build/tasks/build.js b/build/tasks/build.js index 69916bf02..b59ef816e 100644 --- a/build/tasks/build.js +++ b/build/tasks/build.js @@ -65,7 +65,7 @@ module.exports = function( grunt ) { if ( /.\/var\//.test( path.replace( process.cwd(), "" ) ) ) { contents = contents .replace( - /define\([\w\W]*?return/, + /define\(\s*(["'])[\w\W]*?\1[\w\W]*?return/, "var " + ( /var\/([\w-]+)/.exec( name )[ 1 ] ) + " =" -- 2.39.5