RFR: 8010465: Can't enable sjavac when building in jprt.

Tim Bell tim.bell at oracle.com
Mon Apr 8 16:20:01 UTC 2013


Hi Erik:

This looks good - approved.  Sorry for the delay.

Tim


> Reminder that I would like this reviewed.
>
> /Erik
>
> On 2013-03-21 12:49, Erik Joelsson wrote:
>> This patch makes it possible to enable sjavac in jprt runs. This is 
>> achieved by adding -buildenv ENABLE_SJAVAC=true to the jprt command 
>> line.
>>
>> http://cr.openjdk.java.net/~erikj/8010465/webrev.root.01/
>>
>> Doing this uncovered another bug. Adding the keyword "nofile" to the 
>> LOG variable is done to disable logging build output to a file. JPRT 
>> does this on all build runs since it already saves the build output. 
>> The value of LOG is also sent to sjavac to help it filter its output, 
>> but sjavac does not accept "nofile" as input. The logic to remove 
>> "nofile" before sending it to sjavac didn't work and is also fixed in 
>> this patch.
>>
>> The sad news is that all the windows builds currently fail with 
>> sjavac enabled, but having this feature will help us harden sjavac to 
>> the point where we can make the switch.
>>
>> /Erik
>>
>>





More information about the build-dev mailing list