On Tue, 18 Jun 2024 08:32:43 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

> > Does the `throw new Error(f + ": " + message);` should be replaced to 
> > `throw new SkippedException(f + ": " + message);`, to avoid report failure 
> > if a directory cannot be made read-only.
> 
> Yes, it does! That's the whole point. Ironically, I missed that in my diff.

Thanks your advice, the code has been updated according your advice, and has 
been verified on linux and windows.

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

PR Comment: https://git.openjdk.org/jdk/pull/19731#issuecomment-2175555950

Reply via email to