RFR: 8316000: File.setExecutable silently fails if file does not exist [v2]

Brian Burkhalter bpb at openjdk.org
Tue Sep 19 22:08:39 UTC 2023


On Fri, 15 Sep 2023 14:35:43 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> If this code does not interact with ACL-based security, then it's not clear to me that there's anything to be done here.
>
>> If this code does not interact with ACL-based security, then it's not clear to me that there's anything to be done here.
> 
> Right, these methods were created with POSIX file permissions in mind. On the surface, the only result it can return is false (to mean fail) but the concern is that it would be an incompatible change so we need to think a bit more about it.

Commit 08c5fd936e79fe1d08ca5c7fc9cc59a8c71ff555 reverts the impl changes in favor of adjusting the spec to match behavior. This PR will be tagged and a CSR created once there is consensus on the spec update.

Note that the spec of `File.setWritable` is _not_ changed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15673#discussion_r1330754200


More information about the core-libs-dev mailing list