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