Date: Fri, 29 Dec 2023 13:50:34 +0100 From: Dimitry Andric <dim@FreeBSD.org> To: Nuno Teixeira <eduardo@freebsd.org> Cc: tuexen@freebsd.org, FreeBSD CURRENT <freebsd-current@freebsd.org> Subject: Re: devel/nspr: Fails to build on 1500008 5f71f9636efa Message-ID: <C5D01D64-82F6-4713-8156-1D3B68D2E1EA@FreeBSD.org> In-Reply-To: <CAFDf7ULcNYuD4ZAWXCjgbcWBbdmWw2b0O1vb9phqjXCP5VhpJA@mail.gmail.com> References: <CAFDf7U%2BxbTuhS7nUzeZgVknMPyyfYxuakFtrE4=h-1LtFK_Yig@mail.gmail.com> <9EE91B0D-6CFB-464E-AB9C-3A15D77D55FB@freebsd.org> <CAFDf7UJBY9a=SX_iTE2TOxK_r%2BaK1NDizMdebuYeaL6Zb89QbQ@mail.gmail.com> <CAFDf7ULcNYuD4ZAWXCjgbcWBbdmWw2b0O1vb9phqjXCP5VhpJA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
The problem is really that our kernel headers (those under sys/) require = C99. The only thing that = https://cgit.freebsd.org/src/commit/?id=3Da8b70cf26030d68631200619bd1b0ad3= 5b34b6b8 did was move two static inline functions from = sys/netinet/tcp_var.h to sys/netinet/tcp.h, and it looks like the former = header is never directly included by ports. The latter is, so those = ports should be switched to C99, or the sys/netinet/tcp.h header should = be fixed to use __inline instead. -Dimitry > On 29 Dec 2023, at 13:30, Nuno Teixeira <eduardo@freebsd.org> wrote: >=20 > (...) > -ansi =3D=3D Same as -std=3Dc89 >=20 > So it seems correct to remove it when -std=3Dc99 is used. >=20 > Nuno Teixeira <eduardo@freebsd.org> escreveu no dia sexta, 29/12/2023 = =C3=A0(s) 12:03: >=20 >=20 >=20 > I think we have two options: > 1. Build the ports with a C version of at least C99. >=20 > - Adding USE_CSTD=3Dc99 > - Removing -ansi: > --- configure.orig 2023-12-29 11:54:11 UTC > +++ configure > - CFLAGS=3D"$CFLAGS $(DSO_CFLAGS) -ansi -Wall" > + CFLAGS=3D"$CFLAGS $(DSO_CFLAGS) -Wall" >=20 > Fix build. >=20 > Notes: >=20 > Adding USE_CSTD=3Dc99 doesn't fix build by itself like we seen on some = ports that were fixed by it. >=20 > Something have changed from current 1500007 to 1500008. >=20 > 2. Remove the inline from tcp_[gs]et_flags(). >=20 > Best regards > Michael > > --=20 > > Nuno Teixeira > > FreeBSD Committer (ports) >=20 >=20 >=20 > --=20 > Nuno Teixeira > FreeBSD Committer (ports) >=20 >=20 > --=20 > Nuno Teixeira > FreeBSD Committer (ports)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?C5D01D64-82F6-4713-8156-1D3B68D2E1EA>