On Thu, Nov 02, 2017 at 07:08:50PM -0700, Bryan Drewery wrote: > > > > On Nov 2, 2017, at 18:49, Steve Kargl <s...@troutmask.apl.washington.edu> > > wrote: > > > >> On Thu, Nov 02, 2017 at 06:25:24PM -0700, Bryan Drewery wrote: > >> > >> On Nov 2, 2017, at 15:44, Mark Millard <mar...@dsl-only.net> wrote: > >> > >>>> Author: bdrewery > >>>> Date: Thu Nov 2 22:23:00 2017 > >>>> New Revision: 325347 > >>>> URL: > >>>> https://svnweb.freebsd.org/changeset/base/325347 > >>>> > >>>> > >>>> Log: > >>>> Something is very wrong > >> > >> > >> Unfortunately I only test with META_MODE these days which implies > >> -DNO_CLEAN. > > > > You're making changes to the build infrastructure and you're > > not properly testing it before committing? This is beyond > > pointyhat material. > > 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’t recall > from the bus. > > What I missed was the “clean” buildworld because I forgot it even exists. > I’ve wanted to remove it for a year. I also forgot to test buildenv. >
If you did all the above under META_MODE, then no you did not 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 your changes to the build infrastructure. As you have demonstrated, Makefile, Makefile.inc1, and the *.mk files are sufficiently complicated that proper testing should be done, and proper testing means one doesn't takes shortcuts. -- Steve _______________________________________________ freebsd-current@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"
