Hi Ævar,

On Sun, 24 Jan 2021, Ævar Arnfjörð Bjarmason wrote:

> Remove a flag added in my fb95e2e38d (grep: un-break building with
> PCRE >= 8.32 without --enable-jit, 2017-06-01). It's set just below
> USE_LIBPCRE=YesPlease, so it's been redundant since
> e6c531b808 (Makefile: make USE_LIBPCRE=YesPlease mean v2, not v1,
> 2018-03-11).
>
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> ---
>  config.mak.uname | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/config.mak.uname b/config.mak.uname
> index 198ab1e58f..e22d4b6d67 100644
> --- a/config.mak.uname
> +++ b/config.mak.uname
> @@ -664,7 +664,6 @@ else
>  		NO_GETTEXT =
>  		USE_GETTEXT_SCHEME = fallthrough
>  		USE_LIBPCRE= YesPlease
> -		NO_LIBPCRE1_JIT = UnfortunatelyYes

FWIW this is almost precisely what we carry in Git for Windows for a while
(to battle-test it), as
https://github.com/git-for-windows/git/commit/735a58a38d43659cb985d717ec7eda1f26e121e5

The only thing that commit does in addition is to insert the forgotten
space in the line above, between `USE_LIBPCRE` and `=`.

Thanks,
Dscho

>  		NO_CURL =
>  		USE_NED_ALLOCATOR = YesPlease
>  	else
> --
> 2.29.2.222.g5d2a92d10f8
>
>