RFR: JDK-8030781: System.setProperties(null) drops all system properties (RELEASE not set)

[Alan Bateman]

On 08/01/2014 10:17, Erik Joelsson wrote:
>
> So, do you think I should:
>
> 1. Commit the fix without the test and file a followup for a proper 
> test to be implemented.
> 2. Commit the fix with the current test and file a followup for making 
> the test better.
> 3. Fix the test first and then commit the fix.
>
> 3 is certainly best, but I rather don't have the time for it right 
> now. Which of 1 or 2 is preferred?
#2 is okay, it means we'll a minimum test and we can expand on this with 
the follow-up issue.


>
>> "@run main/othervm SetPropertiesNull" will ensure that it runs in its 
>> own VM but it might not be necessary here as system properties is one 
>> of the things that jtreg will reset.
>>
> That should cover us, except that if this test fails, so will jtreg 
> fail with resetting the properties.
I believe it does.

-Alan.