Date: Sat, 28 Mar 2020 19:42:04 -0700 From: Enji Cooper <yaneurabeya@gmail.com> To: Brooks Davis <brooks@freebsd.org> Cc: Enji Cooper <ngie@FreeBSD.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r359385 - head/usr.bin/kyua Message-ID: <2A7EAECB-C11A-458C-9495-BC975300D363@gmail.com> In-Reply-To: <D84490BA-98F4-4421-8453-1FD23936D1A0@gmail.com> References: <202003280108.02S18KWc062782@repo.freebsd.org> <20200328032209.GA40889@spindle.one-eyed-alien.net> <D84490BA-98F4-4421-8453-1FD23936D1A0@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Mar 28, 2020, at 7:37 PM, Enji Cooper <yaneurabeya@gmail.com> = wrote: >=20 >>=20 >> On Mar 27, 2020, at 8:22 PM, Brooks Davis <brooks@freebsd.org = <mailto:brooks@freebsd.org>> wrote: >>=20 >> On Sat, Mar 28, 2020 at 01:08:20AM +0000, Enji Cooper wrote: >>> Author: ngie >>> Date: Sat Mar 28 01:08:20 2020 >>> New Revision: 359385 >>> URL: https://svnweb.freebsd.org/changeset/base/359385 = <https://svnweb.freebsd.org/changeset/base/359385>; >>>=20 >>> Log: >>> Check in the generated copies of the manpages >>>=20 >>> These manpages were meant to be templated once per `configure` run. >>>=20 >>> Given that we're not bound by as many constants, e.g., `--prefix` = isn't >>> generally changing for kyua in the base system, having to generate = the >>> manpages each build seems slightly less than optimal. >>>=20 >>> In the event that one's build environment doesn't define `$SH`, the = build >>> will also fail until this change is introduced. >>>=20 >>> Instead of jumping through hoops dealing with shells or = permissions, let's >>> just cut to the chase and check the generated copies into the = sourcebase >>> under usr.bin/kyua . >>=20 >> This was the wrong fix. Committing generated manpaged was >> 100% the wrong solution and will make extra work in the >> future. The correct fix is almost certainly the one attached to >> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D245086 = <https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D245086>. If = someone >> doesn't beat me to it I will revert you wrong and broken commit along >> with the followups and commit the correct one (this certainly won't = be >> until at least tomorrow). >=20 > I=E2=80=99ll revert my changes. Done in r359417. Thank you, -Enji=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2A7EAECB-C11A-458C-9495-BC975300D363>