Hi,
Janek has just pushed newer (still development, though) version of my patch to dev/frax/colorful-make branch. It is almost completely rewritten and greatly simplified, and fixes several mentioned issues. I'd be happy if you give it a try and share with your feedback.
Especially if you've tried the previous version and encountered some problems, please check whether they are fixed. In particular, color codes should work much better now.
I believe that these changes, with some more
documentation added, will make maintaining this patch easy and almost
no-cost, and keep it very easy to remove(with just few simple sed
replacements) or turn off. David, Phil, Jan - do you think that it makes it more acceptable?
Main differences:
- all logic was moved to make/fancy-printing.make,
- colors got simplified - however it is possible to easily differentiate style of different descriptions, default configuration has only 3 styles: for program name (e.g. g++), for file names, and for rest of descriptions.
- printing descriptions inside rules is in almost all cases is as simply as writing:
$(call PRINT_SMART_DESC,<used program name>)
or
$(call PRINT_GENERIC_DESC,<description>,<optional file name>)
or just
$(call PRINT_SMART_DESC).
No need to specify what action is done - it is deduced from program name.
- echo was changed to printf - possibly it should fix some problems with color codes
- using color codes is now restricted to printing on terminal. While redirecting to file/pipe, they won't be used.
- it is possible to turn echoing back on, with make option VERBOSE_MAKE
- it is possible to turn metafont output back on, with make option VERBOSE_METAFONT (was possible before as well)
- it is possible to turn off "smart" descriptions with option NO_FANCY_PRINTING (which implies VERBOSE_MAKE). Used together with VERBOSE_METAFONT is equivalent to not using this patch at all.
- it is possible to completely turn off color codes with make NO_COLOR option.
Options VERBOSE_MAKE, VERBOSE_METAFONT, NO_FANCY_PRINTING and NO_COLOR are checked for beeing non-empty strings (this is how make's $(if) works), so turning them on means giving them non-empty values (like $make VERBOSE_MAKE=1).
Still pending:
- hiding some more rules,
- making some descriptions more meaningful,
- possibly removing all @-echo-suppressing in favour of .SILENT rule,
- testing,
Greetings,
Franek