RFR: JDK-8148655 LOG=cmdlines and other build-infra fixes
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Mon Feb 1 21:36:42 UTC 2016
On 2016-02-01 16:01, Erik Joelsson wrote:
> Hello,
>
> InitSupport.gmk:
> The comment still refers to ",nofile".
I'll fix.
>
> I would probably have made an effort to replace "$(ECHO) $(call
> ShellQuote, $2) > $(strip $1).cmdline &&" with a call to WriteFile.
I did think about that, yes. However, I didn't figure out a satisfactory
way to solve it. The problem is that WriteFile, on GNU Make < 4, results
in a $(shell) call, which seemed worse than a chained call to echo, when
we already is in a shell command line in a recipe. I wasn't too keen on
creating a WriteFileInRecipe version either. So I couldn't figure out a
way to do that that I was happy with. But I'm open to suggestions (or
even better, working code :-)).
/Magnus
>
> /Erik
>
> On 2016-01-30 11:27, Magnus Ihse Bursie wrote:
>> This is yet another collection of fixes from the build-infra hotspot
>> project forest that has a stand-alone value.
>>
>> The most important change is the support of a new log option,
>> cmdlines. This is, like the old "nofile", an option that can be added
>> to a log level, e.g. "LOG=info,cmdlines" or used standalone
>> "LOG=cmdlines" (in which case the log level stays at default). With
>> this in place, the command line of "important" commands are printed.
>> Examples of "important" commands are compiler and linker calls.
>> Examples of "non-important" commands are "mkdir" or "cat". Note that
>> at this point, not all "important" calls are identified, typically in
>> esoteric stuff like gensrc.
>>
>> Apart from this, a few other changes are also included:
>> * Allow DEBUG_SYMBOLS to be individually turned off (follow up to
>> JDK-8145596)
>> * Support .S assembly files
>> * Expose USERNAME outside configure
>> * Fix broken indentation
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8148655
>> WebRev:
>> http://cr.openjdk.java.net/~ihse/JDK-8148655-LOG-cmdlines-and-misc-fixes/webrev.01
>>
>> /Magnus
>
More information about the build-dev
mailing list