Re: git: a7469c9c0a50 - main - libc: bsort_s() requires both __BSD_VISIBLE and __EXT1_VISIBLE

This is a formal request to revert all commits related to bsort.  It
should not have been committed without much broader discussion and IMO
does not belong in the tree.  It certainly should not be in the tree
under such a generic name.

-- Brooks

On Wed, Apr 19, 2023 at 08:13:18PM +0000, Hans Petter Selasky wrote:
> The branch main has been updated by hselasky:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=a7469c9c0a504a5e6e9b89e148cd78df5e67ff7f
> 
> commit a7469c9c0a504a5e6e9b89e148cd78df5e67ff7f
> Author:     Hans Petter Selasky <hselasky@FreeBSD.org>
> AuthorDate: 2023-04-19 19:37:29 +0000
> Commit:     Hans Petter Selasky <hselasky@FreeBSD.org>
> CommitDate: 2023-04-19 20:12:04 +0000
> 
>     libc: bsort_s() requires both __BSD_VISIBLE and __EXT1_VISIBLE
>     
>     Fixes build of Python:
>     /usr/include/stdlib.h:409:1: error: unknown type name 'errno_t'
>     errno_t  bsort_s(void *, rsize_t, rsize_t,
>     
>     Reported by:    vishwin@
>     MFC after:      1 week
>     Sponsored by:   NVIDIA Networking
>     Differential Revision:  https://reviews.freebsd.org/D36493
> ---
>  include/stdlib.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/stdlib.h b/include/stdlib.h
> index 857092b9053e..3ad28cf68847 100644
> --- a/include/stdlib.h
> +++ b/include/stdlib.h
> @@ -403,12 +403,12 @@ void ignore_handler_s(const char * __restrict, void * __restrict, errno_t);
>  /* K.3.6.3.2 */
>  errno_t	 qsort_s(void *, rsize_t, rsize_t,
>      int (*)(const void *, const void *, void *), void *);
> -#endif /* __EXT1_VISIBLE */
>  
>  #if __BSD_VISIBLE
>  errno_t	 bsort_s(void *, rsize_t, rsize_t,
>      int (*)(const void *, const void *, void *), void *);
>  #endif /* __BSD_VISIBLE */
> +#endif /* __EXT1_VISIBLE */
>  
>  __END_DECLS
>  __NULLABILITY_PRAGMA_POP
>