git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] macos: move NO_REGEX in macOS to config.mak.uname

From
Carlo Marcelo Arenas Belón <carenas@gmail.com>
Date
Apr 25, 2020, 09:15 UTC
Message-ID
<20200425091549.42293-1-carenas@gmail.com>

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(-)
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
-- 
2.26.2.569.g1d74ac4d14
Message 1 of 1 in “macos: move NO_REGEX in macOS to config.mak.uname”
  1. macos: move NO_REGEX in macOS to config.mak.unameCarlo Marcelo Arenas Belón, Apr 25, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.