Date: Mon, 27 Mar 2023 13:59:08 +0100 From: Nuno Teixeira <eduardo@freebsd.org> To: Warner Losh <imp@bsdimp.com> Cc: "Floyd, Paul" <paulf2718@gmail.com>, freebsd-current@freebsd.org Subject: Re: byteswap.h not found on 12 and 13 Message-ID: <CAFDf7UJRNTWNqXqQYXZvqeAG8rdedkYceG9JLS6LDCtKuEA%2B1w@mail.gmail.com> In-Reply-To: <CAFDf7UJ_pB5LGpQU6FFk=W80nGO_d7OypEWNo0BZ5dcKGQUf4g@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> <CANCZdfp=zTD5ctpjcQhDBexWP40d=z0GAd09Qa9eFM4yxXq9RQ@mail.gmail.com> <CANCZdfq2nLt1Bq2Jzn=ivvqaOvF9=y8j=ChZzy6TrE5zAzeFiw@mail.gmail.com> <13BBF767-6CBB-41D5-AD93-6ADD9C417811@FreeBSD.org> <1aa553c9-79b9-30a0-e4ca-95b1057f1d80@gmail.com> <CANCZdfo_okhWq_fks3=s4xYMWc_QcKqxvRgT2ogWsqMZpF25oQ@mail.gmail.com> <CAFDf7UJ_pB5LGpQU6FFk=W80nGO_d7OypEWNo0BZ5dcKGQUf4g@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000a7e86505f7e14f0d Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Fixed at https://cgit.freebsd.org/ports/commit/?id=3Dfb22e1c2e3653558065ffa0509c3c02= 0834723db Thanks all, Nuno Teixeira <eduardo@freebsd.org> escreveu no dia sexta, 24/03/2023 =C3= =A0(s) 11:15: > I'm thinking in the best plan to patch port. > What about patch it for OSVERSION < 14 that will fix 12 and 13 and when > 13.2 is out patching it only for 12. > > Any thoughts? > > Warner Losh <imp@bsdimp.com> escreveu no dia sexta, 24/03/2023 =C3=A0(s) = 09:37: > >> >> >> On Fri, Mar 24, 2023 at 3:23=E2=80=AFAM Floyd, Paul <paulf2718@gmail.com= > wrote: >> >>> >>> A modern and standard solution is std::byteswap. >>> >>> https://en.cppreference.com/w/cpp/numeric/byteswap >>> >>> According to this >>> >>> https://en.cppreference.com/w/cpp/compiler_support#C.2B.2B23_library_fe= atures >>> >>> you will need llvm 14 for that. >>> >> >> Which doesn't help OP since they needed 12 and 13 support for existing >> software. >> Also, that's C++ only, which I don't think helps either. >> >> I'm glad to see some articulated standards here because when I did >> endian.h >> and byteswap.h, it was a minefield of what can or can't be defined in th= e >> face >> of existing use. >> >> Warner >> >> >>> A+ >>> >>> Paul >>> >>> >>> >>> >>> > > -- > Nuno Teixeira > FreeBSD Committer (ports) > --=20 Nuno Teixeira FreeBSD Committer (ports) --000000000000a7e86505f7e14f0d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div>Fixed at <a href=3D"https://cgit.freebsd.org/ports/co= mmit/?id=3Dfb22e1c2e3653558065ffa0509c3c020834723db">https://cgit.freebsd.o= rg/ports/commit/?id=3Dfb22e1c2e3653558065ffa0509c3c020834723db</a></div><di= v><br></div><div>Thanks all,<br></div></div><br><div class=3D"gmail_quote">= <div dir=3D"ltr" class=3D"gmail_attr">Nuno Teixeira <<a href=3D"mailto:e= duardo@freebsd.org">eduardo@freebsd.org</a>> escreveu no dia sexta, 24/0= 3/2023 =C3=A0(s) 11:15:<br></div><blockquote class=3D"gmail_quote" style=3D= "margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-le= ft:1ex"><div dir=3D"ltr"><div>I'm thinking in the best plan to patch po= rt.</div><div>What about patch it for OSVERSION < 14 that will fix 12 an= d 13 and when 13.2 is out patching it only for 12.</div><div><br></div><div= >Any thoughts?<br></div></div><br><div class=3D"gmail_quote"><div dir=3D"lt= r" class=3D"gmail_attr">Warner Losh <<a href=3D"mailto:imp@bsdimp.com" t= arget=3D"_blank">imp@bsdimp.com</a>> escreveu no dia sexta, 24/03/2023 = =C3=A0(s) 09:37:<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"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote"= ><div dir=3D"ltr" class=3D"gmail_attr">On Fri, Mar 24, 2023 at 3:23=E2=80= =AFAM Floyd, Paul <<a href=3D"mailto:paulf2718@gmail.com" target=3D"_bla= nk">paulf2718@gmail.com</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"><br> A modern and standard solution is std::byteswap.<br> <br> <a href=3D"https://en.cppreference.com/w/cpp/numeric/byteswap" rel=3D"noref= errer" target=3D"_blank">https://en.cppreference.com/w/cpp/numeric/byteswap= </a><br> <br> According to this <br> <a href=3D"https://en.cppreference.com/w/cpp/compiler_support#C.2B.2B23_lib= rary_features" rel=3D"noreferrer" target=3D"_blank">https://en.cppreference= .com/w/cpp/compiler_support#C.2B.2B23_library_features</a><br> <br> you will need llvm 14 for that.<br></blockquote><div><br></div><div>Which d= oesn't help OP since they needed 12 and 13 support for existing softwar= e.</div><div>Also, that's C++ only, which I don't think helps eithe= r.</div><div><br></div><div>I'm glad to see some articulated standards = here because when I did endian.h</div><div>and byteswap.h, it was a minefie= ld of what can or can't be defined in the face</div><div>of existing us= e.</div><div><br></div><div>Warner</div><div>=C2=A0</div><blockquote class= =3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg= b(204,204,204);padding-left:1ex"> A+<br> <br> Paul<br> <br> <br> <br> <br> </blockquote></div></div> </blockquote></div><br clear=3D"all"><br><span>-- </span><br><div dir=3D"lt= r"><div dir=3D"ltr"><span style=3D"color:rgb(102,102,102)">Nuno Teixeira<br= >FreeBSD Committer (ports)</span></div></div> </blockquote></div><br clear=3D"all"><br><span class=3D"gmail_signature_pre= fix">-- </span><br><div dir=3D"ltr" class=3D"gmail_signature"><div dir=3D"l= tr"><span style=3D"color:rgb(102,102,102)">Nuno Teixeira<br>FreeBSD Committ= er (ports)</span></div></div> --000000000000a7e86505f7e14f0d--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFDf7UJRNTWNqXqQYXZvqeAG8rdedkYceG9JLS6LDCtKuEA%2B1w>