RFR: 8335912: Add an operation mode to the jar command when extracting to not overwriting existing files [v3]

Henry Jen henryjen at openjdk.org
Wed Oct 2 02:04:35 UTC 2024


On Mon, 30 Sep 2024 15:26:07 GMT, Henry Jen <henryjen at openjdk.org> wrote:

>> Hello Henry, I think this `-k` option help text would need a slight modification. Right now it states that if a file appears more than once in an archive, then this setting this flag will not overwrite the earlier copies. In reality, it doesn't matter how many times the file appears in the archive - even if it appears just once, and if the file is being extracted into a directory which already has the same named file at that path, then this `-k` flag will not overwrite that existing file.
>> 
>> So I think we should reword it to talk about its behaviour in context of some file already existing in the destination directory where the jar contents are being extracted. 
>> 
>> I wonder if we should make a mention that we don't do any case sensitive checks for the existence of the file in the destination directory and instead we use filesystem API to check for existence (which depending on the filesystem may be case insensitive, like macosx).
>
> You have valid point. I refer to the tar man page, and I thought do not overwrite existing files says it. The in particular is just to call out a specific use case.

Please review the update and see if that's good enough.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21141#discussion_r1783755856


More information about the core-libs-dev mailing list