[ https://issues.apache.org/jira/browse/COMPRESS-404?focusedWorklogId=436774&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-436774 ]
ASF GitHub Bot logged work on COMPRESS-404: ------------------------------------------- Author: ASF GitHub Bot Created on: 23/May/20 15:35 Start Date: 23/May/20 15:35 Worklog Time Spent: 10m Work Description: bodewig commented on a change in pull request #97: URL: https://github.com/apache/commons-compress/pull/97#discussion_r429555828 ########## File path: src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java ########## @@ -368,10 +397,74 @@ public TarArchiveEntry(final File file) { * @param fileName the name to be used for the entry. */ public TarArchiveEntry(final File file, final String fileName) { + final String normalizedName = normalizeFileName(fileName, false); + this.file = file.toPath(); + + try { + readFileMode(this.file, normalizedName); + } catch (IOException e) { + // Ignore for backwards compatibility Review comment: Ignoring exceptions here - and documenting that we do - seems to be the best approach. Another alternative would be to not change this constructor's code to use the new methods at all. If we want to invoke the new methods then I'd suggest to set `modTime` and `size` "the old way" before invoking the new methods so we keep the old behaviour if things go wrong in NIO land. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 436774) Time Spent: 2h 50m (was: 2h 40m) > Add Path equivalents to TarArchiveEntry, with methods using File delegating > to equivalents > ------------------------------------------------------------------------------------------ > > Key: COMPRESS-404 > URL: https://issues.apache.org/jira/browse/COMPRESS-404 > Project: Commons Compress > Issue Type: Sub-task > Reporter: Simon Spero > Priority: Minor > Time Spent: 2h 50m > Remaining Estimate: 0h > > (This is a reasonable place to start, as Paths give better access to > tar-relevant metadata on unix system). > Symlink info is easier to obtain > Unix based Paths allow access to useful attributes under "unix:*" > - numeric uid and gid > - string owner and group names > - mtime,ctime,atime > - numeric mode > - numeric dev and inode > - numeric rdev > - Linux, Solaris and Windows allow access to extended attributes > (MacOS X xattrs aren't supported as of jdk 9) -- This message was sent by Atlassian Jira (v8.3.4#803005)