Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 15 Nov 2023 14:59:46 -0700
From:      Warner Losh <imp@bsdimp.com>
To:        Jessica Clarke <jrtc27@freebsd.org>
Cc:        Warner Losh <imp@freebsd.org>,  "src-committers@freebsd.org" <src-committers@freebsd.org>,  "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org>,  "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.org>
Subject:   Re: git: 3fd60a6b73ac - main - awk: Merge upstream 2nd Edition Awk Book
Message-ID:  <CANCZdfq6tWv9skfuA2DAo6AEhJGgFJzVzdWTx3pW8eMaSk-GpQ@mail.gmail.com>
In-Reply-To: <C3D101D8-77BB-4250-B934-18C711BADE46@freebsd.org>
References:  <202311140451.3AE4p5AR014207@gitrepo.freebsd.org> <C3D101D8-77BB-4250-B934-18C711BADE46@freebsd.org>

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

On Wed, Nov 15, 2023 at 9:52=E2=80=AFAM Jessica Clarke <jrtc27@freebsd.org>=
 wrote:

> On 14 Nov 2023, at 04:51, Warner Losh <imp@FreeBSD.org> wrote:
> >
> > The branch main has been updated by imp:
> >
> > URL:
> https://cgit.FreeBSD.org/src/commit/?id=3D3fd60a6b73ac01a72df89751f173970=
fae4cae73
> >
> > commit 3fd60a6b73ac01a72df89751f173970fae4cae73
> > Merge: 194df014feeb 2e406c584fe4
> > Author:     Warner Losh <imp@FreeBSD.org>
> > AuthorDate: 2023-11-02 16:32:01 +0000
> > Commit:     Warner Losh <imp@FreeBSD.org>
> > CommitDate: 2023-11-14 04:49:34 +0000
> >
> >    awk: Merge upstream 2nd Edition Awk Book
> >
> >    Merge in the November 2nd, 2023 version of one true awk.
> >
> >    This brings in Unicode support, CSV support and a number of bug fixe=
s.
> >
> >    Sponsored by:           Netflix
> >    Reviewed by:            delphij
> >    Differential Revision:  https://reviews.freebsd.org/D42447
>
> This seems to have broken the GitHub CI for cross-building; contrast:
>
> https://github.com/freebsd/freebsd-src/actions/runs/6859373313
>
> with:
>
> https://github.com/freebsd/freebsd-src/actions/runs/6859316228
>
> Has this broken makeobjops.awk somehow? If so it should be reproducible
> on FreeBSD with BOOTSTRAP_ALL_TOOLS=3D1, or by rebuilding on an updated
> system. Cross-building will just show it first because we always
> bootstrap awk there, like many other tools.
>

It worked when it was the system awk for me...  But that might not be the
most
helpful observation. Why didn't it work for you/github's ci runs?

makeobjops.awk doesn't have any of the issues that I know are minor changes
in awk. I'll add it to the list of things to look into, though.

I also broke the Jenkins CI run, but not 'make check' for reasons I don't
understand.

Warner

--0000000000002d5458060a380609
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">=
<div dir=3D"ltr" class=3D"gmail_attr">On Wed, Nov 15, 2023 at 9:52=E2=80=AF=
AM Jessica Clarke &lt;<a href=3D"mailto:jrtc27@freebsd.org">jrtc27@freebsd.=
org</a>&gt; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"marg=
in:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1e=
x">On 14 Nov 2023, at 04:51, Warner Losh &lt;imp@FreeBSD.org&gt; wrote:<br>
&gt; <br>
&gt; The branch main has been updated by imp:<br>
&gt; <br>
&gt; URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D3fd60a6b73ac=
01a72df89751f173970fae4cae73" rel=3D"noreferrer" target=3D"_blank">https://=
cgit.FreeBSD.org/src/commit/?id=3D3fd60a6b73ac01a72df89751f173970fae4cae73<=
/a><br>
&gt; <br>
&gt; commit 3fd60a6b73ac01a72df89751f173970fae4cae73<br>
&gt; Merge: 194df014feeb 2e406c584fe4<br>
&gt; Author:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt; AuthorDate: 2023-11-02 16:32:01 +0000<br>
&gt; Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt; CommitDate: 2023-11-14 04:49:34 +0000<br>
&gt; <br>
&gt;=C2=A0 =C2=A0 awk: Merge upstream 2nd Edition Awk Book<br>
&gt; <br>
&gt;=C2=A0 =C2=A0 Merge in the November 2nd, 2023 version of one true awk.<=
br>
&gt; <br>
&gt;=C2=A0 =C2=A0 This brings in Unicode support, CSV support and a number =
of bug fixes.<br>
&gt; <br>
&gt;=C2=A0 =C2=A0 Sponsored by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Net=
flix<br>
&gt;=C2=A0 =C2=A0 Reviewed by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 del=
phij<br>
&gt;=C2=A0 =C2=A0 Differential Revision:=C2=A0 <a href=3D"https://reviews.f=
reebsd.org/D42447" rel=3D"noreferrer" target=3D"_blank">https://reviews.fre=
ebsd.org/D42447</a><br>
<br>
This seems to have broken the GitHub CI for cross-building; contrast:<br>
<br>
<a href=3D"https://github.com/freebsd/freebsd-src/actions/runs/6859373313" =
rel=3D"noreferrer" target=3D"_blank">https://github.com/freebsd/freebsd-src=
/actions/runs/6859373313</a><br>
<br>
with:<br>
<br>
<a href=3D"https://github.com/freebsd/freebsd-src/actions/runs/6859316228" =
rel=3D"noreferrer" target=3D"_blank">https://github.com/freebsd/freebsd-src=
/actions/runs/6859316228</a><br>
<br>
Has this broken makeobjops.awk somehow? If so it should be reproducible<br>
on FreeBSD with BOOTSTRAP_ALL_TOOLS=3D1, or by rebuilding on an updated<br>
system. Cross-building will just show it first because we always<br>
bootstrap awk there, like many other tools.<br></blockquote><div><br></div>=
<div>It worked when it was the system awk for me...=C2=A0 But that might no=
t be the most</div><div>helpful observation. Why didn&#39;t it work for you=
/github&#39;s ci runs?</div><div><br></div><div>makeobjops.awk=C2=A0doesn&#=
39;t have any of the issues that I know are minor changes</div><div>in awk.=
 I&#39;ll add it to the list of things to look into, though.</div><div><br>=
</div><div>I also broke the Jenkins CI run, but not &#39;make check&#39; fo=
r reasons I don&#39;t</div><div>understand.</div><div><br></div><div>Warner=
</div><div><br></div><div><br></div></div></div>

--0000000000002d5458060a380609--



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