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