Date: Thu, 2 Nov 2017 19:41:21 -0700 From: Bryan Drewery <bdrewery@FreeBSD.org> To: sgk@troutmask.apl.washington.edu Cc: 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: <998FF503-D4B0-4AD5-AD55-98680E4D66CA@FreeBSD.org> In-Reply-To: <20171103022327.GA88659@troutmask.apl.washington.edu> 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>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Nov 2, 2017, at 19:23, Steve Kargl <sgk@troutmask.apl.washington.edu> w= rote: >=20 >> On Thu, Nov 02, 2017 at 07:08:50PM -0700, Bryan Drewery wrote: >>=20 >>=20 >>>> On Nov 2, 2017, at 18:49, Steve Kargl <sgk@troutmask.apl.washington.edu= > wrote: >>>>=20 >>>> On Thu, Nov 02, 2017 at 06:25:24PM -0700, Bryan Drewery wrote: >>>>=20 >>>> On Nov 2, 2017, at 15:44, Mark Millard <markmi@dsl-only.net> wrote: >>>>=20 >>>>>> Author: bdrewery >>>>>> Date: Thu Nov 2 22:23:00 2017 >>>>>> New Revision: 325347 >>>>>> URL:=20 >>>>>> https://svnweb.freebsd.org/changeset/base/325347 >>>>>>=20 >>>>>>=20 >>>>>> Log: >>>>>> Something is very wrong >>>>=20 >>>>=20 >>>> Unfortunately I only test with META_MODE these days which implies -DNO_= CLEAN. >>>=20 >>> 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 inst= allworld and installkernel, several xdev and native-xtools, several full DIR= DEPS_BUILD builds and bootstraps, ran subdir builds, ran subdir cleans, test= ed several targets together, ran various special case tests for submakes, pl= ayed around with a ton of MAKEOBJDIRPREFIX cases, handled and tested symlink= ed objdirs special, ran it through my work repro a few times, did special te= sting 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=99t recall fr= om the bus. >>=20 >> What I missed was the =E2=80=9Cclean=E2=80=9D buildworld because I forgot= 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. Are you accusing me of lying? > 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 I did that probably 100 times. And that isn=E2=80=99t even =E2=80=9Cthe prop= er test=E2=80=9D. Both clean and incremental are needed which I did. zfs sna= pshots help a lot there. I just never ran =E2=80=9C_cleanobj=E2=80=9D which d= oes a full tree walk of clean. But I ran make clean in some subdirs many tim= es. > infrastructure. 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. I took 0 shortcuts. As I said I *forgot* that case, among hundreds of cases.= You=E2=80=99re welcome to do this work if you want. I guarantee you would no= t have tested even half of what I tested.=20 Hey can you fix universe to only build clang once please? I=E2=80=99ve been w= orking up to that but I think you=E2=80=99re best to do it. >=20 > --=20 > Steve
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?998FF503-D4B0-4AD5-AD55-98680E4D66CA>