Date: Thu, 11 Sep 2025 10:57:26 -0600 From: Warner Losh <imp@bsdimp.com> To: =?UTF-8?Q?Olivier_Cochard=2DLabb=C3=A9?= <olivier@freebsd.org> Cc: =?UTF-8?Q?Dag=2DErling_Sm=C3=B8rgrav?= <des@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: d20c82507278 - main - Revert "libc: Remove readdir_r(3)" Message-ID: <CANCZdfqqQ-vGTXYQ==J_LD6a2%2Be1GwcGmkJJ5n=4zbyS7Cd9dg@mail.gmail.com> In-Reply-To: <CA%2Bq%2BTcrYcX5eDnKVb2sgvwofUJ55SF6Q0nuEH6L2M4uchSsWhQ@mail.gmail.com> References: <202509111512.58BFCV87088340@gitrepo.freebsd.org> <CA%2Bq%2BTcrYcX5eDnKVb2sgvwofUJ55SF6Q0nuEH6L2M4uchSsWhQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000003702fc063e896f1f Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, Sep 11, 2025 at 10:27=E2=80=AFAM Olivier Cochard-Labb=C3=A9 <olivie= r@freebsd.org> wrote: > > On Thu, Sep 11, 2025 at 8:12=E2=80=AFAM Dag-Erling Sm=C3=B8rgrav <des@fre= ebsd.org> > wrote: > >> The branch main has been updated by des: >> >> URL: >> https://cgit.FreeBSD.org/src/commit/?id=3Dd20c82507278e003f391c3fb04f4e4= 9afd1537ab >> >> commit d20c82507278e003f391c3fb04f4e49afd1537ab >> Author: Dag-Erling Sm=C3=B8rgrav <des@FreeBSD.org> >> AuthorDate: 2025-09-11 15:12:22 +0000 >> Commit: Dag-Erling Sm=C3=B8rgrav <des@FreeBSD.org> >> CommitDate: 2025-09-11 15:12:22 +0000 >> >> Revert "libc: Remove readdir_r(3)" >> >> This reverts commit d549de769055ae6116601e54e4c86dfb3e17f4c4. >> >> > Hi Dag-Erling, > Could you please explain why this change was reversed? > Because the reason was missing from this commit message. > Rust breaks with this change. While rust can be fixed (and will be) since it's doing some sketch things and the fix is to just remove the sketch stuff. However, part of Rust's work flow is to do A/B testing between new runs under test, and older Rust versions. The old binaries work fine, but newly-built copies of the old versions do not. Until those old versions 'age out' of this setup, we'll be stuck with this. Warner --0000000000003702fc063e896f1f Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote g= mail_quote_container"><div dir=3D"ltr" class=3D"gmail_attr">On Thu, Sep 11,= 2025 at 10:27=E2=80=AFAM Olivier Cochard-Labb=C3=A9 <<a href=3D"mailto:= olivier@freebsd.org">olivier@freebsd.org</a>> wrote:<br></div><blockquot= e class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px s= olid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div dir=3D"ltr"><= div style=3D"font-family:"courier new",monospace"><br></div></div= ><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Thu, S= ep 11, 2025 at 8:12=E2=80=AFAM Dag-Erling Sm=C3=B8rgrav <<a href=3D"mail= to:des@freebsd.org" target=3D"_blank">des@freebsd.org</a>> wrote:<br></d= iv><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bord= er-left:1px solid rgb(204,204,204);padding-left:1ex">The branch main has be= en updated by des:<br> <br> URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Dd20c82507278e003f= 391c3fb04f4e49afd1537ab" rel=3D"noreferrer" target=3D"_blank">https://cgit.= FreeBSD.org/src/commit/?id=3Dd20c82507278e003f391c3fb04f4e49afd1537ab</a><b= r> <br> commit d20c82507278e003f391c3fb04f4e49afd1537ab<br> Author:=C2=A0 =C2=A0 =C2=A0Dag-Erling Sm=C3=B8rgrav <des@FreeBSD.org>= <br> AuthorDate: 2025-09-11 15:12:22 +0000<br> Commit:=C2=A0 =C2=A0 =C2=A0Dag-Erling Sm=C3=B8rgrav <des@FreeBSD.org>= <br> CommitDate: 2025-09-11 15:12:22 +0000<br> <br> =C2=A0 =C2=A0 Revert "libc: Remove readdir_r(3)"<br> <br> =C2=A0 =C2=A0 This reverts commit d549de769055ae6116601e54e4c86dfb3e17f4c4.= <br><br></blockquote><div><br></div>Hi Dag-Erling,<br>Could you please expl= ain why this change was reversed?<br>Because the reason was missing from th= is commit message.<br></div></div></blockquote><div><br></div><div>Rust bre= aks with this change. While rust can be fixed (and will be) since it's = doing some sketch things and the fix is to just remove the sketch stuff. Ho= wever, part of Rust's work flow is to do A/B testing between new runs u= nder test, and older Rust versions. The old binaries work fine, but newly-b= uilt copies of the old versions do not. Until those old versions 'age o= ut' of this setup, we'll be stuck with this.</div><div><br></div><d= iv>Warner=C2=A0</div></div></div> --0000000000003702fc063e896f1f--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfqqQ-vGTXYQ==J_LD6a2%2Be1GwcGmkJJ5n=4zbyS7Cd9dg>