Date: Thu, 20 Mar 2025 03:13:28 -0400 From: shurd@FreeBSD.org To: Colin Percival <cperciva@tarsnap.com>, Stephen Hurd <shurd@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 9d0eea9422d0 - main - Make newly POSIX functions visible Message-ID: <8cc8ee54-d6ea-47dc-a22d-57b6b03bda4e@FreeBSD.org> In-Reply-To: <1c4b2e3e-aabc-4fd4-a115-56a9133e9bff@sjh.sh> References: <202503192345.52JNjFxt093046@gitrepo.freebsd.org> <01000195b231d604-edc6ae17-e5d4-4dc6-afd3-995223ecde17-000000@email.amazonses.com> <1c4b2e3e-aabc-4fd4-a115-56a9133e9bff@sjh.sh>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2025-03-20 03:11, Stephen Hurd wrote: > On 2025-03-20 02:16, Colin Percival wrote: >> On 3/19/25 16:45, Stephen Hurd wrote: >>> commit 9d0eea9422d075c8a6924b33161d2d5abfb4072a >>> Author: Stephen Hurd <shurd@FreeBSD.org> >>> AuthorDate: 2025-03-19 23:45:06 +0000 >>> Commit: Stephen Hurd <shurd@FreeBSD.org> >>> CommitDate: 2025-03-19 23:45:06 +0000 >>> >>> Make newly POSIX functions visible >>> Some of the POSIX 202405L functions are already in the >>> system, make >>> them visible when appropriate. >>> Reviewed by: imp >>> MFC after: 1 week >>> Differential Revision: https://reviews.freebsd.org/D47859 >> >> This broke lang/python311: >>> --- Modules/posixmodule.o --- >>> ./Modules/posixmodule.c:12907:9: error: call to undeclared function >>> 'setresuid'; ISO C99 and later do not support implicit function decla >>> rations [-Werror,-Wimplicit-function-declaration] >>> 12907 | if (setresuid(ruid, euid, suid) < 0) >>> | ^ >> I realize this is almost certainly a bug in Python (assuming that these >> functions are available without setting the right POSIX level) but if >> Python has this bug I'm sure other ports will also be affected. Please >> back out this commit and talk to portmgr@ about doing an exp-run to find >> out what needs to be fixed in the ports tree before re-committing this. > > I could also make it #if __POSIX_VISIBLE >= 202405 || __BSD_VISIBLE > > Should fix the problem, but still make them visible for POSIX. https://reviews.freebsd.org/D49421
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?8cc8ee54-d6ea-47dc-a22d-57b6b03bda4e>