darkma773r commented on PR #289: URL: https://github.com/apache/commons-compress/pull/289#issuecomment-1119236042
@garydgregory, this fixes `TarFileTest` on Linux but it's not clear to me if this is the correct fix or if a deeper issue needs to be resolved. The test started failing with the PR for COMPRESS-612 and the extended logic for detecting the XSTAR format [here](https://github.com/apache/commons-compress/blob/afa14ea0fe5c65ab923080ccded72678979563fa/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java#L741). The logic previously evaluated the test tar files as being in POSIX_FORMAT and so appended the '/' character to the end of directory names. It now evaluates to XSTAR and so does not append the '/' character. In order to make the test pass, I changed the requested entry format in the test from BIGNUMBER_STAR to BIGNUMBER_POSIX. -- 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. To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
