Date: Sat, 28 Mar 2020 19:37:13 -0700 From: Enji Cooper <yaneurabeya@gmail.com> To: Brooks Davis <brooks@freebsd.org> Cc: Enji Cooper <ngie@FreeBSD.org>, 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: <D84490BA-98F4-4421-8453-1FD23936D1A0@gmail.com> In-Reply-To: <20200328032209.GA40889@spindle.one-eyed-alien.net> References: <202003280108.02S18KWc062782@repo.freebsd.org> <20200328032209.GA40889@spindle.one-eyed-alien.net>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Mar 27, 2020, at 8:22 PM, Brooks Davis <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 >>=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). I=E2=80=99ll revert my changes. -Enji=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?D84490BA-98F4-4421-8453-1FD23936D1A0>