On Tue, 18 Jun 2024 01:26:33 GMT, SendaoYan <s...@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.

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

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

Reply via email to