Date: Thu, 2 Nov 2017 19:43:20 -0700 From: Bryan Drewery <bdrewery@FreeBSD.org> To: Glen Barber <gjb@FreeBSD.org> Cc: Steve Kargl <sgk@troutmask.apl.washington.edu>, FreeBSD Toolchain <freebsd-toolchain@freebsd.org>, freebsd-hackers <freebsd-hackers@freebsd.org>, FreeBSD Current <freebsd-current@freebsd.org> Subject: Re: Head build unsafe for /etc today Message-ID: <AD087A1B-ABBA-4829-A870-046FB17B3D3A@FreeBSD.org> In-Reply-To: <20171103023651.GC34123@FreeBSD.org> References: <3045EEBF-09E6-4209-B54F-2F95394DBA82@FreeBSD.org> <20171103014907.GA88522@troutmask.apl.washington.edu> <68BECDA4-C182-436E-854C-C3B19ABB4373@FreeBSD.org> <20171103022327.GA88659@troutmask.apl.washington.edu> <20171103023651.GC34123@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Nov 2, 2017, at 19:36, Glen Barber <gjb@FreeBSD.org> wrote: >=20 >> On Thu, Nov 02, 2017 at 07:23:27PM -0700, Steve Kargl wrote: >> On Thu, Nov 02, 2017 at 07:08:50PM -0700, Bryan Drewery wrote: >>>> On Nov 2, 2017, at 18:49, Steve Kargl <sgk@troutmask.apl.washington.edu= > wrote: >>>> You're making changes to the build infrastructure and you're >>>> not properly testing it before committing? This is beyond >>>> pointyhat material.=20 >>>=20 >>> I ran 2 universes, dozens of buildworlds and buildkernels, >>> dozens of installworld and installkernel, several xdev and >>> native-xtools, several full DIRDEPS_BUILD builds and bootstraps, >>> ran subdir builds, ran subdir cleans, tested several targets together, >>> ran various special case tests for submakes, played around with a >>> ton of MAKEOBJDIRPREFIX cases, handled and tested symlinked objdirs >>> special, ran it through my work repro a few times, did special >>> testing in rescue/, and had a volunteer test release. In the process >>> I found a bmake bug, GPL_DTC build bug and several others I don=E2=80=99= t >>> recall from the bus. >>>=20 >>> What I missed was the =E2=80=9Cclean=E2=80=9D buildworld because I forgo= t it >>> even exists. I=E2=80=99ve wanted to remove it for a year. I also forgot t= o >>> test buildenv. >>>=20 >>=20 >> If you did all the above under META_MODE, then no you did not=20 >> buildworld and buildkernel and all the other stuff you claim. >> If your first step isn't 'cd /usr/obj ; rm -rf *' or equivalent >> in whatever jail you use, then you're not properly testing=20 >> your changes to the build infrastructure. >=20 > I did test this, personally, and missed it. To be fair you didn=E2=80=99t test the AUTO_OBJ piece but did test the much m= ore riskier changes to objdir handling. Thanks for that. >> As you have demonstrated, >> Makefile, Makefile.inc1, and the *.mk files are sufficiently=20 >> complicated that proper testing should be done, and proper=20 >> testing means one doesn't takes shortcuts. >>=20 >=20 > Mistakes happen. Things sometimes get accidentally missed. >=20 > Glen >=20
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AD087A1B-ABBA-4829-A870-046FB17B3D3A>