Date: Sun, 28 Apr 2024 14:37:48 +0200 From: Mateusz Piotrowski <0mp@FreeBSD.org> To: Warner Losh <imp@bsdimp.com> Cc: 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> Subject: Re: git: 5626f9e79090 - main - style.9: Document the existence of tools/build/checkstyle9.pl Message-ID: <026867C8-33DC-4555-8B6B-03E0E848510D@FreeBSD.org> In-Reply-To: <CANCZdfokbfo8UsHJL98ZV6jDCKtQS6eFM6-7SJy4t_N2CLpDiA@mail.gmail.com> References: <202404260938.43Q9cSwG055237@gitrepo.freebsd.org> <CANCZdfokbfo8UsHJL98ZV6jDCKtQS6eFM6-7SJy4t_N2CLpDiA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_9FB49A63-E48C-4927-8AF2-0542D31DC5BD Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 Hey Warner, Thanks for taking a look at the patch! > On 26. Apr 2024, at 15:30, Warner Losh <imp@bsdimp.com> wrote: > On Fri, Apr 26, 2024, 3:38=E2=80=AFAM Mateusz Piotrowski = <0mp@freebsd.org <mailto:0mp@freebsd.org>> wrote: >> The branch main has been updated by 0mp: >>=20 >> URL: = https://cgit.FreeBSD.org/src/commit/?id=3D5626f9e790901999def12040a60746b0= ceeb9b95 = <https://cgit.freebsd.org/src/commit/?id=3D5626f9e790901999def12040a60746b= 0ceeb9b95> [...] >>=20 >> style.9: Document the existence of tools/build/checkstyle9.pl = <http://checkstyle9.pl/>; [...] >> share/man/man9/style.9 | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) [...] >> @@ -904,6 +904,10 @@ instead of the older >> .Fn CTASSERT . >> .Sh FILES >> .Bl -tag -width indent >> +.It Pa /usr/src/tools/build/checkstyle9.pl <http://checkstyle9.pl/>; >> +A script to check for violations of >=20 >=20 > Egregious violations and there are some false positives still. checkstyle9 is already warning against those imperfections. I assumed = that there is no need to warn against those in the manual page as well. = I can add additional disclaimers if you think they are desired :) Best, Mateusz Piotrowski --Apple-Mail=_9FB49A63-E48C-4927-8AF2-0542D31DC5BD Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=utf-8 <html><head><meta http-equiv=3D"content-type" content=3D"text/html; = charset=3Dutf-8"></head><body style=3D"overflow-wrap: break-word; = -webkit-nbsp-mode: space; line-break: after-white-space;">Hey = Warner,<div><br></div><div>Thanks for taking a look at the patch!<br = id=3D"lineBreakAtBeginningOfMessage"><div><br><blockquote = type=3D"cite"><div>On 26. Apr 2024, at 15:30, Warner Losh = <imp@bsdimp.com> wrote:</div><div><div dir=3D"auto" = style=3D"caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: = 12px; font-style: normal; font-variant-caps: normal; font-weight: 400; = letter-spacing: normal; text-align: start; text-indent: 0px; = text-transform: none; white-space: normal; word-spacing: 0px; = -webkit-text-stroke-width: 0px; text-decoration: none;"><div><div = class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Fri, Apr = 26, 2024, 3:38=E2=80=AFAM Mateusz Piotrowski <<a = href=3D"mailto:0mp@freebsd.org">0mp@freebsd.org</a>> = wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin: 0px = 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; = border-left-color: rgb(204, 204, 204); padding-left: 1ex;">The branch = main has been updated by 0mp:<br><br>URL:<span = class=3D"Apple-converted-space"> </span><a = href=3D"https://cgit.freebsd.org/src/commit/?id=3D5626f9e790901999def12040= a60746b0ceeb9b95" rel=3D"noreferrer noreferrer" = target=3D"_blank">https://cgit.FreeBSD.org/src/commit/?id=3D5626f9e7909019= 99def12040a60746b0ceeb9b95</a><br></blockquote></div></div></div></div></b= lockquote>[...]<br><blockquote type=3D"cite"><div><div dir=3D"auto" = style=3D"caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: = 12px; font-style: normal; font-variant-caps: normal; font-weight: 400; = letter-spacing: normal; text-align: start; text-indent: 0px; = text-transform: none; white-space: normal; word-spacing: 0px; = -webkit-text-stroke-width: 0px; text-decoration: none;"><div><div = class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"margin: = 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; = border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><br> = <span class=3D"Apple-converted-space"> </span>style.9: = Document the existence of tools/build/<a href=3D"http://checkstyle9.pl/" = rel=3D"noreferrer noreferrer" = target=3D"_blank">checkstyle9.pl</a><br></blockquote></div></div></div></d= iv></blockquote>[...]<br><blockquote type=3D"cite"><div><div dir=3D"auto" = style=3D"caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: = 12px; font-style: normal; font-variant-caps: normal; font-weight: 400; = letter-spacing: normal; text-align: start; text-indent: 0px; = text-transform: none; white-space: normal; word-spacing: 0px; = -webkit-text-stroke-width: 0px; text-decoration: none;"><div><div = class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"margin: = 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; = border-left-color: rgb(204, 204, 204); padding-left: = 1ex;"> share/man/man9/style.9 | 6 +++++-<br> 1 file changed, 5 = insertions(+), 1 = deletion(-)<br></blockquote></div></div></div></div></blockquote>[...]<br>= <blockquote type=3D"cite"><div><div dir=3D"auto" style=3D"caret-color: = rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: = normal; font-variant-caps: normal; font-weight: 400; letter-spacing: = normal; text-align: start; text-indent: 0px; text-transform: none; = white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; = text-decoration: none;"><div><div class=3D"gmail_quote"><blockquote = class=3D"gmail_quote" style=3D"margin: 0px 0px 0px 0.8ex; = border-left-width: 1px; border-left-style: solid; border-left-color: = rgb(204, 204, 204); padding-left: 1ex;">@@ -904,6 +904,10 @@ instead of = the older<br> .Fn CTASSERT .<br> .Sh FILES<br> .Bl -tag = -width indent<br>+.It Pa /usr/src/tools/build/<a = href=3D"http://checkstyle9.pl/" rel=3D"noreferrer noreferrer" = target=3D"_blank">checkstyle9.pl</a><br>+A script to check for = violations of<br></blockquote></div></div><div dir=3D"auto"><br></div><div= dir=3D"auto">Egregious violations and there are some false positives = still.</div></div></div></blockquote><br></div><div>checkstyle9 is = already warning against those imperfections. I assumed that there is no = need to warn against those in the manual page as well. I can add = additional disclaimers if you think they are desired = :)</div><div><br></div><div>Best,</div><div>Mateusz = Piotrowski</div><br></div></body></html>= --Apple-Mail=_9FB49A63-E48C-4927-8AF2-0542D31DC5BD--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?026867C8-33DC-4555-8B6B-03E0E848510D>