Date: Wed, 6 Dec 2023 22:50:21 -0700 From: Warner Losh <imp@bsdimp.com> To: Alan Somers <asomers@freebsd.org> Cc: Zhenlei Huang <zlei@freebsd.org>, src-committers <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>, li-Wen Hsu <lwhsu@freebsd.org> Subject: Re: git: 6b96125afdf2 - main - cap_net.3: remove a copypasta Message-ID: <CANCZdfpzfKdULQ6TjiOt7JaqbghZKjQ9wWphPExMADpuFbQ3qQ@mail.gmail.com> In-Reply-To: <CAOtMX2gvox5qfm8dASpD-4Z3qAqM8v13BRUBs8fKWa98Gm_hpQ@mail.gmail.com> References: <202312061652.3B6GqOuf068349@gitrepo.freebsd.org> <44E8C9F3-84DD-4E51-B2A4-FB56E2A08F30@FreeBSD.org> <CAOtMX2gvox5qfm8dASpD-4Z3qAqM8v13BRUBs8fKWa98Gm_hpQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000d05322060be50b48 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, Dec 6, 2023, 8:59 PM Alan Somers <asomers@freebsd.org> wrote: > On Wed, Dec 6, 2023 at 6:55=E2=80=AFPM Zhenlei Huang <zlei@freebsd.org> w= rote: > > > > > > > > > On Dec 7, 2023, at 12:52 AM, Alan Somers <asomers@freebsd.org> wrote: > > > > > > The branch main has been updated by asomers: > > > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3D6b96125afdf245ae61dd82b59891ad0= d6aab0066 > > > > > > commit 6b96125afdf245ae61dd82b59891ad0d6aab0066 > > > Author: Alan Somers <asomers@FreeBSD.org> > > > AuthorDate: 2023-12-05 23:23:29 +0000 > > > Commit: Alan Somers <asomers@FreeBSD.org> > > > CommitDate: 2023-12-06 16:51:37 +0000 > > > > > > cap_net.3: remove a copypasta > > > > > > This line appears to have been copied from cap_sysctl.3. While I'= m > > > here, reorder and reword the description of cap_net_limit a bit. > > > > > > [skip ci] > > > > I guess we can 'skip ci' implicitly for document or typo changes. > > Can we? The skipping logic is builtin to Jenkins, Github Workflows, > and Cirrus. I don't think it would be easy to program any of those to > detect which changes can be safely skipped. > The message currently is a nearly nop for our setup. Jenkins runs arent triggered by a commit. Cirrus CI already can't maje it more than a few days into the month. Commits aren't gated into main by CI. So IMHO, it just adds noise. Let's drop it until it has an actual beneficial effect. Metadata like that doesn't really belong in the git log. As this situation changes, we can reevaluate... Again just my opinion... Warner -Alan > --000000000000d05322060be50b48 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" = class=3D"gmail_attr">On Wed, Dec 6, 2023, 8:59 PM Alan Somers <<a href= =3D"mailto:asomers@freebsd.org">asomers@freebsd.org</a>> wrote:<br></div= ><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1= px #ccc solid;padding-left:1ex">On Wed, Dec 6, 2023 at 6:55=E2=80=AFPM Zhen= lei Huang <<a href=3D"mailto:zlei@freebsd.org" target=3D"_blank" rel=3D"= noreferrer">zlei@freebsd.org</a>> wrote:<br> ><br> ><br> ><br> > > On Dec 7, 2023, at 12:52 AM, Alan Somers <<a href=3D"mailto:as= omers@freebsd.org" target=3D"_blank" rel=3D"noreferrer">asomers@freebsd.org= </a>> wrote:<br> > ><br> > > The branch main has been updated by asomers:<br> > ><br> > > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D6b96125= afdf245ae61dd82b59891ad0d6aab0066" rel=3D"noreferrer noreferrer" target=3D"= _blank">https://cgit.FreeBSD.org/src/commit/?id=3D6b96125afdf245ae61dd82b59= 891ad0d6aab0066</a><br> > ><br> > > commit 6b96125afdf245ae61dd82b59891ad0d6aab0066<br> > > Author:=C2=A0 =C2=A0 =C2=A0Alan Somers <asomers@FreeBSD.org>= ;<br> > > AuthorDate: 2023-12-05 23:23:29 +0000<br> > > Commit:=C2=A0 =C2=A0 =C2=A0Alan Somers <asomers@FreeBSD.org>= ;<br> > > CommitDate: 2023-12-06 16:51:37 +0000<br> > ><br> > >=C2=A0 =C2=A0 cap_net.3: remove a copypasta<br> > ><br> > >=C2=A0 =C2=A0 This line appears to have been copied from cap_sysct= l.3.=C2=A0 While I'm<br> > >=C2=A0 =C2=A0 here, reorder and reword the description of cap_net_= limit a bit.<br> > ><br> > >=C2=A0 =C2=A0 [skip ci]<br> ><br> > I guess we can 'skip ci' implicitly for document or typo chang= es.<br> <br> Can we?=C2=A0 =C2=A0The skipping logic is builtin to Jenkins, Github Workfl= ows,<br> and Cirrus.=C2=A0 I don't think it would be easy to program any of thos= e to<br> detect which changes can be safely skipped.<br></blockquote></div></div><di= v dir=3D"auto"><br></div><div dir=3D"auto">The message currently is a nearl= y nop for our setup. Jenkins runs arent triggered by a commit. Cirrus CI al= ready can't maje it more than a few days into the month. Commits aren&#= 39;t gated into main by CI. So IMHO, it just adds noise. Let's drop it = until it has an actual beneficial effect. Metadata like that doesn't re= ally belong in the git log.</div><div dir=3D"auto"><br></div><div dir=3D"au= to">As this situation changes, we can reevaluate...</div><div dir=3D"auto">= <br></div><div dir=3D"auto">Again just my opinion...</div><div dir=3D"auto"= ><br></div><div dir=3D"auto">Warner</div><div dir=3D"auto"><br></div><div d= ir=3D"auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" st= yle=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> -Alan<br> </blockquote></div></div></div> --000000000000d05322060be50b48--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpzfKdULQ6TjiOt7JaqbghZKjQ9wWphPExMADpuFbQ3qQ>