[ 
https://issues.apache.org/jira/browse/COMPRESS-404?focusedWorklogId=430224&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-430224
 ]

ASF GitHub Bot logged work on COMPRESS-404:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 04/May/20 14:23
            Start Date: 04/May/20 14:23
    Worklog Time Spent: 10m 
      Work Description: theobisproject commented on a change in pull request 
#97:
URL: https://github.com/apache/commons-compress/pull/97#discussion_r419474533



##########
File path: 
src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java
##########
@@ -384,12 +430,49 @@ public TarArchiveEntry(final File file, final String 
fileName) {
         } else {
             this.mode = DEFAULT_FILE_MODE;
             this.linkFlag = LF_NORMAL;
-            this.size = file.length();
+            try {
+                this.size = Files.size(file);
+            } catch (IOException e) {
+                e.printStackTrace();

Review comment:
       That is exactly where I wan't the feedback how to best handle it for 
backward compatibility 😉 See the second part in the description




----------------------------------------------------------------
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:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 430224)
    Time Spent: 40m  (was: 0.5h)

> 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: 40m
>  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)

Reply via email to