macos: move NO_REGEX in macOS to config.mak.uname
diff mbox series

Message ID 20200425091549.42293-1-carenas@gmail.com
State New
Headers show
Series
  • macos: move NO_REGEX in macOS to config.mak.uname
Related show

Commit Message

Carlo Marcelo Arenas Belón April 25, 2020, 9:15 a.m. UTC
29de20504e (Makefile: fix default regex settings on Darwin, 2013-05-11)
added this flag to the Makefile since Mac OS X 10.8 at the time didn't
have a working REG_EXTENDED|REG_NEWLINE

move it to config.mak.uname where all the other system specific settings
are kept since e1b6dbb554 (Makefile: hoist uname autodetection to
config.mak.uname, 2013-01-03) and update the rule so it only applies to
those known broken versions as the problem was most likely fixed in the
next version when Apple imported TRE on their libc.

Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com>
---
 Makefile         | 1 -
 config.mak.uname | 3 +++
 2 files changed, 3 insertions(+), 1 deletion(-)

Patch
diff mbox series

diff --git a/Makefile b/Makefile
index dc356ce4dd..7e3fc7ca79 100644
--- a/Makefile
+++ b/Makefile
@@ -1308,7 +1308,6 @@  ifeq ($(uname_S),Darwin)
 		APPLE_COMMON_CRYPTO = YesPlease
 		COMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO
 	endif
-	NO_REGEX = YesPlease
 	PTHREAD_LIBS =
 endif
 
diff --git a/config.mak.uname b/config.mak.uname
index 0ab8e00938..f89acdd15f 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -125,6 +125,9 @@  ifeq ($(uname_S),Darwin)
 		HAVE_GETDELIM = YesPlease
 	endif
 	NO_MEMMEM = YesPlease
+	ifeq ($(shell test `expr "$(uname_R)" : '\([0-9]*\)\.'` -le 12 && echo 1),1)
+		NO_REGEX = YesPlease
+	endif
 	USE_ST_TIMESPEC = YesPlease
 	HAVE_DEV_TTY = YesPlease
 	COMPAT_OBJS += compat/precompose_utf8.o