RFR: 8212703: Remove sun.java2d.fontpath property from java launcher code

Roger Riggs Roger.Riggs at oracle.com
Mon Dec 10 17:10:28 UTC 2018


Looks fine,

Thanks for the test update.

Roger



On 12/09/2018 02:59 AM, Philip Race wrote:
> Here is an update with a Java-only test, using ProcessBuilder.
>
> http://cr.openjdk.java.net/~prr/8212703.1/index.html
>
> -phil.
>
>
> On 12/1/18, 9:27 AM, Alan Bateman wrote:
>> On 30/11/2018 23:11, Roger Riggs wrote:
>>> Hi Phil,
>>>
>>> That looks fine.
>>>
>>> Too bad it introduces a new shell test, we're trying to get rid of 
>>> them.
>>> The other alternative would have the test program needed to set the 
>>> environment
>>> and launch a java program.
>>>
>> The implementation look good to me too and I agree with Roger that we 
>> shouldn't be introducing a shell test for this. In this case, it 
>> looks like you just need a /othervm test that checks for the env 
>> variable and property.
>>
>> -Alan



More information about the core-libs-dev mailing list