[PATCH v3 modular] build.sh: better integrate --autoresume and -n

Trevor Woerner twoerner at gmail.com
Tue Jan 31 12:34:03 PST 2012


On Tue, Jan 31, 2012 at 3:09 PM, Gaetan Nadon <memsize at videotron.ca> wrote:
> A couple of minor problems. I wouldn't rock the code too much as we can
> live with them. I am prepared to accept the patch as it is.

Excellent, thanks for the review and testing :-)

> If autoresume reaches the last module in the autoresume file, it will
> get built anyway, whether it has previously passed or failed.

It was coded that way on purpose to preserve the existing behaviour of
--autoresume (it always assumes the last module in the built list
failed), but I admit even I found it weird (no matter how the last
module did it always has an implicit "FAIL: " status, but I didn't
think you'd approve if the behaviour changed). The "fix" for this
would entail figuring out if there is a next module to build and
finding out what that next module might be. That isn't an easy thing
to do. It is easier when a --modfile is provided, but only slightly
so.


More information about the xorg-devel mailing list