Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 24 Mar 2023 11:15:54 +0000
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:  <CAFDf7UJ_pB5LGpQU6FFk=W80nGO_d7OypEWNo0BZ5dcKGQUf4g@mail.gmail.com>
In-Reply-To: <CANCZdfo_okhWq_fks3=s4xYMWc_QcKqxvRgT2ogWsqMZpF25oQ@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>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000e77c0605f7a384c9
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

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_fea=
tures
>>
>> 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 the
> face
> of existing use.
>
> Warner
>
>
>> A+
>>
>> Paul
>>
>>
>>
>>
>>

--=20
Nuno Teixeira
FreeBSD Committer (ports)

--000000000000e77c0605f7a384c9
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>I&#39;m thinking in the best plan to patch port.</div=
><div>What about patch it for OSVERSION &lt; 14 that will fix 12 and 13 and=
 when 13.2 is out patching it only for 12.</div><div><br></div><div>Any tho=
ughts?<br></div></div><br><div class=3D"gmail_quote"><div dir=3D"ltr" class=
=3D"gmail_attr">Warner Losh &lt;<a href=3D"mailto:imp@bsdimp.com">imp@bsdim=
p.com</a>&gt; 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-l=
eft: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 &lt;<a hre=
f=3D"mailto:paulf2718@gmail.com" target=3D"_blank">paulf2718@gmail.com</a>&=
gt; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0=
px 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&#39;t help OP since they needed 12 and 13 support for existing softwar=
e.</div><div>Also, that&#39;s C++ only, which I don&#39;t think helps eithe=
r.</div><div><br></div><div>I&#39;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&#39;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 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>

--000000000000e77c0605f7a384c9--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFDf7UJ_pB5LGpQU6FFk=W80nGO_d7OypEWNo0BZ5dcKGQUf4g>