From nobody Wed Apr 19 20:29:52 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Q1spq5Tvxz46QPY; Wed, 19 Apr 2023 20:29:55 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: from spindle.one-eyed-alien.net (spindle.one-eyed-alien.net [199.48.129.229]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4Q1spn5R5Tz4LC3; Wed, 19 Apr 2023 20:29:53 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Authentication-Results: mx1.freebsd.org; none Received: by spindle.one-eyed-alien.net (Postfix, from userid 3001) id 3FEA93C0199; Wed, 19 Apr 2023 20:29:52 +0000 (UTC) Date: Wed, 19 Apr 2023 20:29:52 +0000 From: Brooks Davis To: Hans Petter Selasky Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: a7469c9c0a50 - main - libc: bsort_s() requires both __BSD_VISIBLE and __EXT1_VISIBLE Message-ID: References: <202304192013.33JKDIrM070521@gitrepo.freebsd.org> List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In-Reply-To: <202304192013.33JKDIrM070521@gitrepo.freebsd.org> X-Rspamd-Queue-Id: 4Q1spn5R5Tz4LC3 X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:36236, ipnet:199.48.128.0/22, country:US] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-ThisMailContainsUnwantedMimeParts: N 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: >=20 > URL: https://cgit.FreeBSD.org/src/commit/?id=3Da7469c9c0a504a5e6e9b89e148= cd78df5e67ff7f >=20 > commit a7469c9c0a504a5e6e9b89e148cd78df5e67ff7f > Author: Hans Petter Selasky > AuthorDate: 2023-04-19 19:37:29 +0000 > Commit: Hans Petter Selasky > CommitDate: 2023-04-19 20:12:04 +0000 >=20 > libc: bsort_s() requires both __BSD_VISIBLE and __EXT1_VISIBLE > =20 > 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, > =20 > 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(-) >=20 > 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 */ > =20 > #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 */ > =20 > __END_DECLS > __NULLABILITY_PRAGMA_POP >=20