Re: [RFC PATCH 1/2] grep: fallback to interpreter if JIT fails with pcre1
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Dec 10, 2018, 06:48 UTC
- Message-ID
- <xmqqpnu9khc3.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <20181209230024.43444-2-carenas@gmail.com>
Carlo Marcelo Arenas Belón <carenas@gmail.com> writes:
> JIT support was added to 8.20 but the interface we rely on is only > enabled after 8.32 so try to make the message clearer.
Could you add some word before 8.20 and 8.32 (e.g. "pcre library version 8.20", if that is what you are referring to, and if 8.32 is also taken from the same context, adding such phrase to clarify the context only to 8.20 is sufficient)?
Show 6 quoted lines
> in systems where there are restrictions against creating executable > pages programatically (like OpenBSD, NetBSD, macOS or seLinux) JIT > will fail, resulting in a error message to the user. > > Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> > ---
The change to the code looked sensible.