On Wed, 29 Oct 2025 18:35:46 GMT, Phil Race <[email protected]> wrote:

>> Then It might be better to drop it, or at least mark it as manual. The 
>> current issue with the test is that it modifies the system and JDK, which 
>> could affect other tests running in parallel.
>> 
>> For the JDK, we can create a copy of TESTJAVA and run the test there. 
>> However, we can’t do the same for the system. The situation could become 
>> worse if the tests are terminated midway, as the cleanup step wouldn’t be 
>> executed and the system could remain misconfigured.
>
> A test that modifies the windows registry is going to be a problem. I would 
> just scrap the test.
> I wouldn't even want to run such a test manually.

As far as I can see, the 'manual' is not intended for manual environment 
preparation/cleanup. So I am inclined to remove the test entirely.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/27551#discussion_r2474666196

Reply via email to