Date: Fri, 24 Mar 2023 03:01:43 -0600 From: Warner Losh <imp@bsdimp.com> To: Yuri <yuri@aetern.org> Cc: Nuno Teixeira <eduardo@freebsd.org>, FreeBSD CURRENT <freebsd-current@freebsd.org>, FreeBSD Mailing List <freebsd-ports@freebsd.org> Subject: Re: byteswap.h not found on 12 and 13 Message-ID: <CANCZdfp=zTD5ctpjcQhDBexWP40d=z0GAd09Qa9eFM4yxXq9RQ@mail.gmail.com> In-Reply-To: <CANCZdfrvHQTZqgGdZjm5Z_fa53g92c_6EHtKZpyRAsKwRsDcLw@mail.gmail.com> References: <CAFDf7UKq3bMgd3XA0=U10%2BLhOCY7fZCHds9DdBcrSuNidOityw@mail.gmail.com> <CANCZdfpT3nbZ%2BzHHtdsMDa88JsWr7Frw%2BRPGcy6B=P60DZL2_g@mail.gmail.com> <CAFDf7UL=av0KDAw2O3Fmf=x5wThezBokCQ2KHtW6wOZnn43bxQ@mail.gmail.com> <ef18fee5-358b-3c19-467b-2fd980dff797@aetern.org> <CANCZdfrvHQTZqgGdZjm5Z_fa53g92c_6EHtKZpyRAsKwRsDcLw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000000956f505f7a1a5a3
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
And I just looked it up... unlike endian.h, byteswap.h is not in the draft
I have of issue 8 posix (due out this year or next, give or take).
Warner
On Fri, Mar 24, 2023 at 2:57=E2=80=AFAM Warner Losh <imp@bsdimp.com> wrote:
> Yea, I just committed this
>
> commit 1761b09bf42d2842e82c1ac614c23d31c4d4c0dc
> Author: Warner Losh <imp@FreeBSD.org>
> Date: Fri Jan 20 16:33:37 2023 -0700
>
> byteswap.h: Add a glibc/linux compatible byteswap.h
>
> For endian.h to work instead of sys/endian.h, some software needs
> byteswap.h available. It must define {__,}byteswap_{16,32,64}.
> Included sys/_endian.h to get an appropriate __byteswap16, etc
> and defines the new macros in terms of them. Enhance _endian.h
> to allow it to be included from here too.
>
> Sponsored by: Netflix
> Reviewed by: markj
> Differential Revision: https://reviews.freebsd.org/D32051
>
> which I MFC'd to 13, but after 13.1R and this isn't a good EN candidate.
> I've not bothered with stable/12 because I'm lame and have stopped MFCing
> stuff to it except critical fixes.
>
> Long term: this will make life better. Short term, it's less good since w=
e
> have to still do old-school compatibility.
>
> I likely should write a man page for endian.h and byteswap.h
>
> Warner
>
> On Fri, Mar 24, 2023 at 2:48=E2=80=AFAM Yuri <yuri@aetern.org> wrote:
>
>> Nuno Teixeira wrote:
>> > Hello Warner,
>> >
>> > My poudriere jails:
>> > ---
>> > 124amd64 12.4-RELEASE-p2 amd64 http 2023-03-17
>> > 08:04:15 /usr/local/poudriere/jails/124amd64
>> > 124i386 12.4-RELEASE-p2 i386 http 2023-03-17
>> > 08:04:38 /usr/local/poudriere/jails/124i386
>> > 131amd64 13.1-RELEASE-p7 amd64 http 2023-03-17
>> > 08:05:03 /usr/local/poudriere/jails/131amd64
>> > ---
>> >
>> > The strange thing is that 5.08 (old version) have #include <byteswap.h=
>
>> > too and compiles fine|:
>> > |
>> > https://github.com/sflow/sflowtool/issues/41
>> > <https://github.com/sflow/sflowtool/issues/41>;
>>
>> 5.08 seems to have it commented out:
>>
>> sflowtool-5.08/src/sflowtool.c:
>>
>> /*
>> #ifdef DARWIN
>> #include <architecture/byte_order.h>
>> #define bswap_16(x) NXSwapShort(x)
>> #define bswap_32(x) NXSwapInt(x)
>> #else
>> #include <byteswap.h>
>> #endif
>> */
>>
>>
>> > Warner Losh <imp@bsdimp.com <mailto:imp@bsdimp.com>> escreveu no dia
>> > sexta, 24/03/2023 =C3=A0(s) 08:30:
>> >
>> >
>> >
>> > On Fri, Mar 24, 2023, 9:23 AM Nuno Teixeira <eduardo@freebsd.org
>> > <mailto:eduardo@freebsd.org>> wrote:
>> >
>> > Hello all,
>> >
>> > I'm getting a file not found on 12 and 13 compiling
>> > net/sflowtool latest update:
>> > It compile fine on 14.
>> >
>> > I've searched 14 src and found:
>> > ---
>> > ./include/byteswap.h
>> > ./contrib/ofed/include/byteswap.h
>> > ./contrib/llvm-project/libcxx/include/__bit/byteswap.h
>> > ---
>> > Any clues?
>> >
>> >
>> >
>> > I added it a short time ago. I thought I mfc'd it to 13 but not 12=
.
>> > How recent a 13? It's a non standard glibc extension that may be i=
n
>> > the next posix standard though. I've not looked at the draft for i=
t
>> > yet to see if it complies or not.
>> >
>> > Warner
>> >
>> > Thanks,
>> >
>> > ---
>> > =3D=3D=3D> Building for sflowtool-6.01
>> > --- all ---
>> > /usr/bin/make all-recursive
>> > --- all-recursive ---
>> > Making all in src
>> > --- sflowtool.o ---
>> > cc -DHAVE_CONFIG_H -I. -I.. -O2 -pipe
>> > -fstack-protector-strong -fno-strict-aliasing -MT sflowtool.o
>> > -MD -MP -MF .deps/sflowtool.Tpo -c -o sflowtool.o sflowtool.c
>> > sflowtool.c:32:10: fatal error: 'byteswap.h' file not found
>> > #include <byteswap.h>
>> > ^~~~~~~~~~~~
>> > 1 error generated.
>> > *** [sflowtool.o] Error code 1
>> > ---
>> >
>> >
>> > --
>> > Nuno Teixeira
>> > FreeBSD Committer (ports)
>> >
>> >
>> >
>> > --
>> > Nuno Teixeira
>> > FreeBSD Committer (ports)
>>
>>
--0000000000000956f505f7a1a5a3
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
<div dir=3D"ltr"><div>And I just looked it up... unlike endian.h, byteswap.=
h is not in the draft I have of issue 8 posix (due out this year or next, g=
ive or take).</div><div><br></div><div>Warner</div><br><div class=3D"gmail_=
quote"><div dir=3D"ltr" class=3D"gmail_attr">On Fri, Mar 24, 2023 at 2:57=
=E2=80=AFAM Warner Losh <<a href=3D"mailto:imp@bsdimp.com">imp@bsdimp.co=
m</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin=
:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"=
><div dir=3D"ltr">Yea, I just committed this<div><br></div><div>commit 1761=
b09bf42d2842e82c1ac614c23d31c4d4c0dc<br>Author: Warner Losh <imp@FreeBSD=
.org><br>Date: =C2=A0 Fri Jan 20 16:33:37 2023 -0700<br><br>=C2=A0 =C2=
=A0 byteswap.h: Add a glibc/linux compatible byteswap.h<br><br>=C2=A0 =C2=
=A0 For endian.h to work instead of sys/endian.h, some software needs<br>=
=C2=A0 =C2=A0 byteswap.h available. It must define {__,}byteswap_{16,32,64}=
.<br>=C2=A0 =C2=A0 Included sys/_endian.h to get an appropriate __byteswap1=
6, etc<br>=C2=A0 =C2=A0 and defines the new macros in terms of them. Enhanc=
e _endian.h<br>=C2=A0 =C2=A0 to allow it to be included from here too.<br><=
br>=C2=A0 =C2=A0 Sponsored by: =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 Netflix<b=
r>=C2=A0 =C2=A0 Reviewed by: =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0markj=
<br>=C2=A0 =C2=A0 Differential Revision: =C2=A0<a href=3D"https://reviews.f=
reebsd.org/D32051" target=3D"_blank">https://reviews.freebsd.org/D32051</a>=
<br></div><div><br></div><div>which I MFC'd to 13, but after 13.1R and =
this isn't a good EN candidate. I've not bothered with stable/12 be=
cause I'm lame and have stopped MFCing stuff to it except critical fixe=
s.</div><div><br></div><div>Long term: this will make life better. Short te=
rm, it's less good since we have to still do old-school compatibility.<=
/div><div><br></div><div>I likely should write a man page for endian.h and =
byteswap.h</div><div><br></div><div>Warner</div></div><br><div class=3D"gma=
il_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Fri, Mar 24, 2023 at 2:4=
8=E2=80=AFAM Yuri <<a href=3D"mailto:yuri@aetern.org" target=3D"_blank">=
yuri@aetern.org</a>> wrote:<br></div><blockquote class=3D"gmail_quote" s=
tyle=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);pad=
ding-left:1ex">Nuno Teixeira wrote:<br>
> Hello Warner,<br>
> <br>
> My poudriere jails:<br>
> ---<br>
> 124amd64 =C2=A0 12.4-RELEASE-p2 =C2=A0 =C2=A0 =C2=A0amd64 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 http =C2=A0 =C2=A0 =C2=A0 =C2=A0 2023-03-17<br>
> 08:04:15 /usr/local/poudriere/jails/124amd64<br>
> 124i386 =C2=A0 =C2=A012.4-RELEASE-p2 =C2=A0 =C2=A0 =C2=A0i386 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0http =C2=A0 =C2=A0 =C2=A0 =C2=A0 2023-03-17<br>
> 08:04:38 /usr/local/poudriere/jails/124i386<br>
> 131amd64 =C2=A0 13.1-RELEASE-p7 =C2=A0 =C2=A0 =C2=A0amd64 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 http =C2=A0 =C2=A0 =C2=A0 =C2=A0 2023-03-17<br>
> 08:05:03 /usr/local/poudriere/jails/131amd64<br>
> ---<br>
> <br>
> The strange thing is that 5.08 (old version) have #include <byteswa=
p.h><br>
> too and compiles fine|:<br>
> |<br>
> <a href=3D"https://github.com/sflow/sflowtool/issues/41" rel=3D"norefe=
rrer" target=3D"_blank">https://github.com/sflow/sflowtool/issues/41</a><br=
>
> <<a href=3D"https://github.com/sflow/sflowtool/issues/41" rel=3D"no=
referrer" target=3D"_blank">https://github.com/sflow/sflowtool/issues/41</a=
>><br>
<br>
5.08 seems to have it commented out:<br>
<br>
sflowtool-5.08/src/sflowtool.c:<br>
<br>
/*<br>
#ifdef DARWIN<br>
#include <architecture/byte_order.h><br>
#define bswap_16(x) NXSwapShort(x)<br>
#define bswap_32(x) NXSwapInt(x)<br>
#else<br>
#include <byteswap.h><br>
#endif<br>
*/<br>
<br>
<br>
> Warner Losh <<a href=3D"mailto:imp@bsdimp.com" target=3D"_blank">im=
p@bsdimp.com</a> <mailto:<a href=3D"mailto:imp@bsdimp.com" target=3D"_bl=
ank">imp@bsdimp.com</a>>> escreveu no dia<br>
> sexta, 24/03/2023 =C3=A0(s) 08:30:<br>
> <br>
> <br>
> <br>
>=C2=A0 =C2=A0 =C2=A0On Fri, Mar 24, 2023, 9:23 AM Nuno Teixeira <<a =
href=3D"mailto:eduardo@freebsd.org" target=3D"_blank">eduardo@freebsd.org</=
a><br>
>=C2=A0 =C2=A0 =C2=A0<mailto:<a href=3D"mailto:eduardo@freebsd.org" t=
arget=3D"_blank">eduardo@freebsd.org</a>>> wrote:<br>
> <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Hello all,<br>
> <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0I'm getting a file not found on 1=
2 and 13 compiling<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0net/sflowtool latest update:<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0It compile fine on 14.<br>
> <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0I've searched 14 src and found:<b=
r>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0---<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0./include/byteswap.h<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0./contrib/ofed/include/byteswap.h<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0./contrib/llvm-project/libcxx/include=
/__bit/byteswap.h<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0---<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Any clues?<br>
> <br>
> <br>
> <br>
>=C2=A0 =C2=A0 =C2=A0I added it a short time ago. I thought I mfc'd =
it to 13 but not 12.<br>
>=C2=A0 =C2=A0 =C2=A0How recent a 13? It's a non standard glibc exte=
nsion that may be in<br>
>=C2=A0 =C2=A0 =C2=A0the next posix standard though. I've not looked=
at the draft for it<br>
>=C2=A0 =C2=A0 =C2=A0yet to see if it complies or not.<br>
> <br>
>=C2=A0 =C2=A0 =C2=A0Warner<br>
> <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Thanks,<br>
> <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0---<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=3D=3D=3D> =C2=A0Building for sflo=
wtool-6.01<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0--- all ---<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0/usr/bin/make =C2=A0all-recursive<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0--- all-recursive ---<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Making all in src<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0--- sflowtool.o ---<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0cc -DHAVE_CONFIG_H -I. -I.. =C2=A0 =
=C2=A0 =C2=A0-O2 -pipe<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=C2=A0-fstack-protector-strong -fno-s=
trict-aliasing -MT sflowtool.o<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0-MD -MP -MF .deps/sflowtool.Tpo -c -o=
sflowtool.o sflowtool.c<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0sflowtool.c:32:10: fatal error: '=
byteswap.h' file not found<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0#include <byteswap.h><br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0^~~=
~~~~~~~~~<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A01 error generated.<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0*** [sflowtool.o] Error code 1<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0---<br>
> <br>
> <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0-- <br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Nuno Teixeira<br>
>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0FreeBSD Committer (ports)<br>
> <br>
> <br>
> <br>
> -- <br>
> Nuno Teixeira<br>
> FreeBSD Committer (ports)<br>
<br>
</blockquote></div>
</blockquote></div></div>
--0000000000000956f505f7a1a5a3--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfp=zTD5ctpjcQhDBexWP40d=z0GAd09Qa9eFM4yxXq9RQ>