[10] Request for review and approval: JDK-8177770: Need more precise control on build system logging

[David Holmes]

On 30/03/2017 5:10 PM, Magnus Ihse Bursie wrote:
> On 2017-03-30 03:04, David Holmes wrote:
>> Hi Alexey,
>>
>> Can't really review the details of this but wanted to ensure that this
>> gets tested on all platforms before being pushed. Internally use JPRT
>> and "-testset hotspot", but you should also reach out to the AIX-port
>> folk to ensure it works as expected there too.
>
> David,
>
> What are your worries here? There are no platform-dependent changes,
> apart from the actual logging script which might not work properly if
> your local copy of time/flock is non-compliant. (We try to test for
> this, at least for "time".) Unless you're running with LOG=profile, this
> patch affects nothing.

Just want to make sure it doesn't cause any unexpected failures on less 
mainstream OS.

> If the profiling does not work properly on e.g. AIX, then some of the
> AIX folks would need to supply an additional patch to update
> shell-profiler.sh.

Sure. But it would be nice to not break things first. Some contributors 
continually pull OpenJDK changes and run them through their own build 
and test systems.

Thanks,
David

> /Magnus
>>
>> Thanks,
>> David
>>
>> On 30/03/2017 2:21 AM, alexey semenyuk wrote:
>>> Please review and approve enhancement for JDK10.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8177770
>>> Webrev: http://cr.openjdk.java.net/~asemenyuk/8177770/webrev.00/
>>>
>>> Thanks,
>>> Alexey
>>>
>