Date: Thu, 25 Jan 2024 02:22:24 +0000 From: Jessica Clarke <jrtc27@freebsd.org> To: Warner Losh <imp@bsdimp.com> Cc: Ed Maste <emaste@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: 820401bf1769 - main - nl_types.h: explicitly include cdefs.h Message-ID: <549CBD3B-51AA-4BDE-A987-8E98BF1EE8C0@freebsd.org> In-Reply-To: <CANCZdfoY-kBd9KRcRbWDOPCcu5kGC_z1zQPkNF%2B5=HEa%2BUJ46Q@mail.gmail.com> References: <202401242205.40OM5Jiq042676@gitrepo.freebsd.org> <CANCZdfoY-kBd9KRcRbWDOPCcu5kGC_z1zQPkNF%2B5=HEa%2BUJ46Q@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 25 Jan 2024, at 01:24, Warner Losh <imp@bsdimp.com> wrote: >=20 > This was one of the things i wanted to comment on the that review. >=20 > This change should be done as a special Linux sys/param.h that = includes ctypes.h and then includes the next sys/param.h. >=20 > A large number of the changes from that review are similar and i = wanted yo think through the right solution. >=20 > This is fragile: I'm likely to remove it again because it violates = style(9) and has no docs around it. Agreed this is wrong. If it were in tools/build/cross-build I wouldn=E2=80= =99t care about that, but this is base, and the whole point of the cross-build infrastructure is to not have to change base, especially system headers, wherever possible. This is the perfect example of the kind of thing that the cross-build compat headers are meant to fix. So yeah, please revert this someone, else I will. Jess > Warner >=20 > On Wed, Jan 24, 2024, 3:05=E2=80=AFPM Ed Maste <emaste@freebsd.org> = wrote: > The branch main has been updated by emaste: >=20 > URL: = https://cgit.FreeBSD.org/src/commit/?id=3D820401bf176996e41091d2e4190aded1= 2a966b1d >=20 > commit 820401bf176996e41091d2e4190aded12a966b1d > Author: Val Packett <val@packett.cool> > AuthorDate: 2024-01-16 18:42:26 +0000 > Commit: Ed Maste <emaste@FreeBSD.org> > CommitDate: 2024-01-24 21:59:18 +0000 >=20 > nl_types.h: explicitly include cdefs.h >=20 > This file uses __BEGIN_DECLS/__END_DECLS. When bootstrapping from > Linux/musl, they were missing. >=20 > Sponsored by: https://www.patreon.com/valpackett > Pull Request: https://github.com/freebsd/freebsd-src/pull/1066 > --- > include/nl_types.h | 1 + > 1 file changed, 1 insertion(+) >=20 > diff --git a/include/nl_types.h b/include/nl_types.h > index 120d1911631f..6ba15132126d 100644 > --- a/include/nl_types.h > +++ b/include/nl_types.h > @@ -34,6 +34,7 @@ > #ifndef _NL_TYPES_H_ > #define _NL_TYPES_H_ >=20 > +#include <sys/cdefs.h> > #include <sys/types.h> >=20 > #ifdef _NLS_PRIVATE
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?549CBD3B-51AA-4BDE-A987-8E98BF1EE8C0>