From: Junio C Hamano <gitster@pobox•com>
To: git@vger•kernel.org
Cc: Carlo Arenas <carenas@gmail•com>, Brad Smith <brad@comstyle•com>,
"brian m. carlson" <sandals@crustytoothpaste•net>
Subject: [PATCH v3] config.mak.uname: update settings for FreeBSD
Date: Thu, 12 Jun 2025 14:51:02 -0700 [thread overview]
Message-ID: <xmqqv7p0bpdl.fsf_-_@gitster.g> (raw)
In-Reply-To: <aEsE8S90fJSr9Or5@fruit.crustytoothpaste.net> (brian m. carlson's message of "Thu, 12 Jun 2025 16:48:49 +0000")
Even though FreeBSD 6 introduced memmem(), the implementation in
that version was buggy and not performant until FreeBSD 12, FreeBSD
11.4 (the last version of FreeBSD 11) went end of life in September
2021, so nobody should be using it since it hasn't had security
support since then. And memmem() has even been functional (but
slow) since FreeBSD 11.0, and 10.4 went EOL in 2018. So users
shouldn't actually be experiencing any actual functionality problems
since then.
Let's draw the line to require FreeBSD 12 or newer (but we do not
officially document it or enforce it by breaking build when compiled
on older versions, at least not yet), which allows us to drop the
special casing of FreeBSD 4.x and rely on platform implementation of
memmem() unconditionally.
Signed-off-by: Brad Smith <brad@comstyle•com>
[jc: log message with help from info by brian carlson]
Signed-off-by: Junio C Hamano <gitster@pobox•com>
---
* So,... earlier I mentioned about officially documenting the EoL
timeline for various platform support, but without any official
one that documents the decision in one place, how about a commit
with a detailed log like this one, which I stole from brian?
No code changes since v2; only the log message talks more about
where we draw the line and why.
config.mak.uname | 6 ------
1 file changed, 6 deletions(-)
diff --git a/config.mak.uname b/config.mak.uname
index b12d4e168a..5d18d92cb1 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -273,16 +273,10 @@ ifeq ($(uname_S),FreeBSD)
ifeq ($(firstword $(subst -, ,$(uname_R))),10.1)
OLD_ICONV = YesPlease
endif
- NO_MEMMEM = YesPlease
BASIC_CFLAGS += -I/usr/local/include
BASIC_LDFLAGS += -L/usr/local/lib
DIR_HAS_BSD_GROUP_SEMANTICS = YesPlease
USE_ST_TIMESPEC = YesPlease
- ifeq ($(shell expr "$(uname_R)" : '4\.'),2)
- PTHREAD_LIBS = -pthread
- NO_UINTMAX_T = YesPlease
- NO_STRTOUMAX = YesPlease
- endif
PYTHON_PATH = /usr/local/bin/python
PERL_PATH = /usr/local/bin/perl
HAVE_PATHS_H = YesPlease
--
2.50.0-rc2-215-g56f75d5edf
next prev parent reply other threads:[~2025-06-12 21:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-12 13:52 [PATCH v2] config.mak.uname: update settings for FreeBSD Carlo Arenas
2025-06-12 16:48 ` brian m. carlson
2025-06-12 21:31 ` Carlo Marcelo Arenas Belón
2025-06-12 21:51 ` Junio C Hamano [this message]
2025-06-12 22:30 ` [PATCH v3] " Carlo Marcelo Arenas Belón
2025-06-12 22:37 ` Junio C Hamano
2025-07-02 9:37 ` [PATCH v4 0/2] " Carlo Marcelo Arenas Belón
2025-07-02 9:37 ` [PATCH v4 1/2] config.mak.uname: set NO_MEMMEM only for functional version Carlo Marcelo Arenas Belón
2025-07-02 9:37 ` [PATCH v4 2/2] build: retire NO_UINTMAX_T Carlo Marcelo Arenas Belón
2025-07-02 16:21 ` [PATCH v4 0/2] config.mak.uname: update settings for FreeBSD Junio C Hamano
2025-06-12 16:52 ` [PATCH v2] " Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqv7p0bpdl.fsf_-_@gitster.g \
--to=gitster@pobox$(echo .)com \
--cc=brad@comstyle$(echo .)com \
--cc=carenas@gmail$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=sandals@crustytoothpaste$(echo .)net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox