Re: [PATCH v1] string.h: Unconditionally expose C23 functions

["A. Wilcox" <AWilcox@Wilcox-Tech.com>]

On Jul 11, 2025, at 15:50, James Y Knight <jyknight@google.com> wrote:
> 
> On Fri, Jul 11, 2025 at 2:55 PM Thorsten Glaser <tg@mirbsd.de> wrote:
> > I’d ask to at least
> > consider it a little longer given you have actual user reports
> > that they have to patch out things like this downstream.
> 
> Do you have examples of what, concretely, was broken and required the patch? The commit message for the change that ifdef'd away aligned_alloc, at_quick_exit, and quick_exit provides no info about why you found it important to patch downstream -- only that you did so.

NSS, cURL, Boost, and multiple Perl pods all define their own `quick_exit` methods or variables.  Debian Code Search may find more; this is just what I have local in /usr/src.

It is also a hard-fail from PCTS (the POSIX Conformance Test Suite) for an implementation to have visible definitions in the main namespace that are not defined by the standard.

It also ensures that -std is specified properly by upstream packages.  We have already reported multiple instances to upstreams of their not specifying the correct standard version in their build recipes, causing spurious failures.  I can find some of these reports if you need them.

Best,
-Anna