You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Torsten Curdt (JIRA)" <ji...@apache.org> on 2016/01/06 13:39:39 UTC

[jira] [Created] (COMPRESS-328) TarArchiveEntry preserveLeadingSlashes has no effect on setName

Torsten Curdt created COMPRESS-328:
--------------------------------------

             Summary: TarArchiveEntry preserveLeadingSlashes has no effect on setName
                 Key: COMPRESS-328
                 URL: https://issues.apache.org/jira/browse/COMPRESS-328
             Project: Commons Compress
          Issue Type: Improvement
            Reporter: Torsten Curdt
            Priority: Minor


We've run into an inconsistency with the TarArchiveEntry at jdeb.

  https://github.com/tcurdt/jdeb/issues/217

You can create a `TarArchiveEntry(String name, boolean preserveLeadingSlashes)` but the `preserveLeadingSlashes` is only applied in the constructor.

https://github.com/apache/commons-compress/blob/master/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java#L392

I am proposing to turn `preserveLeadingSlashes` into a read-only property and use the value on `setName()`, too (instead of just false).

This has some implications and maybe some backwards compatibility issues - but even then I think it would be the right thing to do.

I am happy to make the change but thought to discuss this first.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)