484363 - pymake should support -k (--keep-going)

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Arpad Borsos [:Swatinem]]

From gnu makes manpage:
       -k, --keep-going
            Continue  as  much  as  possible after an error.  While the target
            that failed, and those that depend on it, cannot  be  remade,  the
            other dependencies of these targets can be processed all the same.

pymake currently fails when passing -k:

make.py: error: no such option: -k

It should either start to support this flag or just ignore it instead of failing.

Comment 1

[Arpad Borsos [:Swatinem]]

I have a clone of pymake here: http://hg.swatinem.de/pymake/
currently adds support for #T grep-for: "text" and adds a test for -k behavior.
I'm still trying to understand how pymake actually works and I may make -k work sometime.

Comment 2

[Arpad Borsos [:Swatinem]]

Attachment: support for -k and #T grep-for

This patch includes changes to runtests to support #T grep-for: syntax, a test for keep-going behavior and the implementation for --keep-going.

I wonder if there is any easier way to pass that flag on as to create a new Makefile member variable.

Comments much appreciated.

Comment 3

[Benjamin Smedberg]

http://hg.mozilla.org/users/bsmedberg_mozilla.com/pymake/rev/50d8e87e8af3

Intermediate changes totally hosed your patch, plus there were details being missed in the parallel case. I think both of those have been resolved in my rework.