Re: git: 296c05f7212c - main - libutil++: Include <cerrno> in stringf.cc

On 8/2/26 05:10, Robert Clausecker wrote:
> The branch main has been updated by fuz:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=296c05f7212c37bd6e6bf7e42af11d42a275d493
> 
> commit 296c05f7212c37bd6e6bf7e42af11d42a275d493
> Author:     Faraz Vahedi <[email protected]>
> AuthorDate: 2026-05-11 06:05:18 +0000
> Commit:     Robert Clausecker <[email protected]>
> CommitDate: 2026-08-02 11:17:45 +0000
> 
>     libutil++: Include <cerrno> in stringf.cc
>     
>     stringf.cc uses errno and related macros without including <cerrno>.
>     
>     Their availability is guaranteed only when the corresponding header
>     is included; transitive exposure is implementation-defined.
>     
>     Modern libc++ has been progressively reducing incidental transitive
>     includes as part of its header removal policy (see LLVM libc++ Header
>     Removal Policy and D132284), making such dependencies brittle.
>     
>     This change includes <cerrno> explicitly to make the dependency
>     well-defined. No functional or behavioural change intended.
>     
>     Approved by:    fuz
>     Signed-off-by:  Faraz Vahedi <[email protected]>
>     Pull-Request:   https://github.com/freebsd/freebsd-src/pull/2188
> ---
>  lib/libutil++/libutil++.hh | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/libutil++/libutil++.hh b/lib/libutil++/libutil++.hh
> index 60e6b3fc5fde..3ba46ed5c506 100644
> --- a/lib/libutil++/libutil++.hh
> +++ b/lib/libutil++/libutil++.hh
> @@ -13,6 +13,7 @@
>  #include <netdb.h>
>  #include <unistd.h>
>  
> +#include <cerrno>
>  #include <cstdarg>
>  #include <cstdio>
>  #include <cstdlib>
> 
> 

Will this be MFC'd ?

-- 
===
Mark Millard
marklmi at yahoo.com