> On Nov 2, 2017, at 19:36, Glen Barber <g...@freebsd.org> wrote:
>> 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 <s...@troutmask.apl.washington.edu>
>>>> 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.
> I did test this, personally, and missed it.
To be fair you didn’t test the AUTO_OBJ piece but did test the much more
riskier changes to objdir handling. Thanks for that.
>> 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.
> Mistakes happen. Things sometimes get accidentally missed.
email@example.com mailing list
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"