[GitHub] commons-compress pull request #38: Testing hudson/jenkins (added contributor...

2017-06-20 Thread sesuncedu
Github user sesuncedu closed the pull request at: https://github.com/apache/commons-compress/pull/38 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #36: COMPRESS-410 Remove Non-NIO character set...

2017-06-20 Thread sesuncedu
Github user sesuncedu closed the pull request at: https://github.com/apache/commons-compress/pull/36 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #36: COMPRESS-410 Remove Non-NIO character set...

2017-06-20 Thread sesuncedu
GitHub user sesuncedu reopened a pull request: https://github.com/apache/commons-compress/pull/36 COMPRESS-410 Remove Non-NIO character set encoders. As a special case, the UTF-8 encoder will replace malformed / unmappable input with '?'. This behavior is required for compatibil

[GitHub] commons-compress pull request #37: COMPRESS-410 - cherry picked to fix mergi...

2017-06-20 Thread sesuncedu
Github user sesuncedu closed the pull request at: https://github.com/apache/commons-compress/pull/37 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #37: COMPRESS-410 - cherry picked to fix mergi...

2017-06-20 Thread sesuncedu
GitHub user sesuncedu reopened a pull request: https://github.com/apache/commons-compress/pull/37 COMPRESS-410 - cherry picked to fix merging issues Remove non-NIO character set encoders. Fixes rebasing related issues in previous PR You can merge this pull request into

[GitHub] commons-compress pull request #38: Testing hudson/jenkins (added contributor...

2017-06-19 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/38 Testing hudson/jenkins (added contributor line to pom.xml so that there would be a change) They've been failing PRs on the checkout of master. Lets try this. You can merge this pull req

[GitHub] commons-compress pull request #37: COMPRESS-410 - cherry picked to fix mergi...

2017-06-18 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/37 COMPRESS-410 - cherry picked to fix merging issues Remove non-NIO character set encoders. Fixes rebasing related issues in previous PR You can merge this pull request into a

[GitHub] commons-compress pull request #36: COMPRESS-410 Remove Non-NIO character set...

2017-06-18 Thread sesuncedu
Github user sesuncedu closed the pull request at: https://github.com/apache/commons-compress/pull/36 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #35: Compress 412

2017-06-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/35 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #33: add-some-Unit-Tests Added some Unit Tests...

2017-06-17 Thread bodewig
Github user bodewig commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/33#discussion_r122573708 --- Diff: src/test/java/org/apache/commons/compress/compressors/xz/XZCompressorOutputStreamTest.java --- @@ -0,0 +1,51 @@ +/* + * Licensed

[GitHub] commons-compress pull request #33: add-some-Unit-Tests Added some Unit Tests...

2017-06-17 Thread sesuncedu
Github user sesuncedu commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/33#discussion_r122569687 --- Diff: src/test/java/org/apache/commons/compress/compressors/xz/XZCompressorOutputStreamTest.java --- @@ -0,0 +1,51 @@ +/* + * License

[GitHub] commons-compress pull request #33: add-some-Unit-Tests Added some Unit Tests...

2017-06-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/33 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #36: COMPRESS-410 Remove Non-NIO character set...

2017-06-16 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/36 COMPRESS-410 Remove Non-NIO character set encoders. As a special case, the UTF-8 encoder will replace malformed / unmappable input with '?'. This behavior is required for compatibilit

[GitHub] commons-compress pull request #35: Compress 412

2017-06-16 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-compress/pull/35 Compress 412 Fixed Jira issue Compress 412: https://issues.apache.org/jira/browse/COMPRESS-412 You can merge this pull request into a Git repository by running: $ git pull https:/

[GitHub] commons-compress pull request #34: COMPRESS-405 : Add javadoc for FixedLengt...

2017-06-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/34 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #34: COMPRESS-405 : Add javadoc for FixedLengt...

2017-06-15 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/34 COMPRESS-405 : Add javadoc for FixedLengthBlockOutputStream. Signed-off-by: Simon Spero You can merge this pull request into a Git repository by running: $ git pull https://github.c

[GitHub] commons-compress pull request #28: COMPRESS-405 Create Fixed Length Block Ou...

2017-06-15 Thread sesuncedu
Github user sesuncedu commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/28#discussion_r122259069 --- Diff: src/main/java/org/apache/commons/compress/utils/FixedLengthBlockOutputStream.java --- @@ -0,0 +1,227 @@ +/* + * Licensed to the

[GitHub] commons-compress pull request #28: COMPRESS-405 Create Fixed Length Block Ou...

2017-06-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #28: COMPRESS-405 Create Fixed Length Block Ou...

2017-06-15 Thread bodewig
Github user bodewig commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/28#discussion_r122244704 --- Diff: src/main/java/org/apache/commons/compress/utils/FixedLengthBlockOutputStream.java --- @@ -0,0 +1,227 @@ +/* + * Licensed to the A

[GitHub] commons-compress pull request #32: Enable maven cache for in travis config

2017-06-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/32 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #33: add-some-Unit-Tests Added some Unit Tests...

2017-06-13 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-compress/pull/33 add-some-Unit-Tests Added some Unit Tests to increase code coverage. I have added some Unit Tests to increase code coverage and want to contribute them. You can merge this pull request

[GitHub] commons-compress pull request #32: Enable maven cache for in travis config

2017-06-13 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/32 Enable maven cache for in travis config Signed-off-by: Simon Spero You can merge this pull request into a Git repository by running: $ git pull https://github.com/sesuncedu/com

[GitHub] commons-compress pull request #27: COMPRESS-400

2017-06-12 Thread sesuncedu
Github user sesuncedu closed the pull request at: https://github.com/apache/commons-compress/pull/27 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #31: COMPRESS-400 It should be possible for us...

2017-06-12 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/31 COMPRESS-400 It should be possible for users to create and access extra PAX headers to tar archives Add extra header map to tar archive entry. Move PAX header processing code from Tar

[GitHub] commons-compress pull request #29: COMPRESS-406 BUILDING.md is missing licen...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/29 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #30: COMPRESS-407 - Validate Block and Record ...

2017-06-11 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/30 COMPRESS-407 - Validate Block and Record Sizes and use 512 blocks if none given. This PR includes the commits for COMPRESS-405 and COMPRESS-406. The new content is https://g

[GitHub] commons-compress pull request #29: COMPRESS-406 BUILDING.md is missing licen...

2017-06-11 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/29 COMPRESS-406 BUILDING.md is missing license header Signed-off-by: Simon Spero You can merge this pull request into a Git repository by running: $ git pull https://github.com/sesunce

[GitHub] commons-compress pull request #28: COMPRESS-405 Create Fixed Length Block Ou...

2017-06-11 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/28 COMPRESS-405 Create Fixed Length Block OutputStream / WriteableByteChannel This PR provides a new class that is an OutputStream and WritableByteChannel, and which supports writing to

[GitHub] commons-compress pull request #27: COMPRESS-400

2017-06-05 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/27 COMPRESS-400 Add extra header map to tar archive entry. Move PAX header processing code from TarArchiveInputStream to TarAchiveEntry. Use same code for processing user supplied e

[GitHub] commons-compress pull request #26: COMPRESS-399 OSGI package versions are ov...

2017-06-02 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/26 COMPRESS-399 OSGI package versions are overly pessimistic, except when they're overly optimisic - Add packageinfo files for per-package version settings. - Initialize to 1.14

[GitHub] commons-compress pull request #25: actually use commons.module.name

2017-05-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/25 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #25: actually use commons.module.name

2017-05-29 Thread bodewig
GitHub user bodewig opened a pull request: https://github.com/apache/commons-compress/pull/25 actually use commons.module.name Tiny change to trigger https://builds.apache.org/job/Commons-Compress%20PullRequest%20Builder/ You can merge this pull request into a Git repository by run

[GitHub] commons-compress pull request #24: COMPRESS-391: Allow alignment on zip cont...

2017-05-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/24 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #24: COMPRESS-391: Allow alignment on zip cont...

2017-05-04 Thread kvr000
GitHub user kvr000 opened a pull request: https://github.com/apache/commons-compress/pull/24 COMPRESS-391: Allow alignment on zip content You can merge this pull request into a Git repository by running: $ git pull https://github.com/kvr000/commons-compress feature/COMPRESS-3

[GitHub] commons-compress pull request #22: COMPRESS-390: Expose stream offsets and s...

2017-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/22 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #23: Add Brotli Support

2017-05-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/23 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #:

2017-05-02 Thread garydgregory
Github user garydgregory commented on the pull request: https://github.com/apache/commons-compress/commit/522b6c67c28f2bcb0d9b09fd47b10449cb1c578c#commitcomment-21978708 In pom.xml: In pom.xml on line 74: I'm pretty sure you do not need to state the scope to `compile`. We do no

[GitHub] commons-compress pull request #23: Add Brotli Support

2017-05-01 Thread bodewig
Github user bodewig commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/23#discussion_r114257946 --- Diff: pom.xml --- @@ -68,6 +68,12 @@ jar, tar, zip, dump, 7z, arj. test + org.brotli + dec +

[GitHub] commons-compress pull request #23: Add Brotli Support

2017-05-01 Thread bodewig
Github user bodewig commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/23#discussion_r114257473 --- Diff: pom.xml --- @@ -85,6 +91,13 @@ jar, tar, zip, dump, 7z, arj. ${powermock.version} test + +

[GitHub] commons-compress pull request #23: Add Brotli Support

2017-05-01 Thread bindul
Github user bindul commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/23#discussion_r114257132 --- Diff: pom.xml --- @@ -68,6 +68,12 @@ jar, tar, zip, dump, 7z, arj. test + org.brotli + dec +

[GitHub] commons-compress pull request #23: Add Brotli Support

2017-05-01 Thread pmouawad
GitHub user pmouawad opened a pull request: https://github.com/apache/commons-compress/pull/23 Add Brotli Support Fix for: https://issues.apache.org/jira/browse/COMPRESS-392 You can merge this pull request into a Git repository by running: $ git pull https://github.com/pmou

[GitHub] commons-compress pull request #22: COMPRESS-390: Expose stream offsets and s...

2017-04-25 Thread kvr000
GitHub user kvr000 opened a pull request: https://github.com/apache/commons-compress/pull/22 COMPRESS-390: Expose stream offsets and size Pull request for https://issues.apache.org/jira/browse/COMPRESS-390 You can merge this pull request into a Git repository by running: $ git

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/20 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/21 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-25 Thread tballison
Github user tballison commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r113181027 --- Diff: src/main/java/org/apache/commons/compress/compressors/lzma/LZMACompressorInputStream.java --- @@ -56,7 +56,7 @@ public LZMACompressorIn

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-25 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r113178004 --- Diff: src/main/java/org/apache/commons/compress/compressors/lzma/LZMACompressorInputStream.java --- @@ -56,7 +56,7 @@ public LZMACompressorInputS

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-25 Thread tballison
Github user tballison commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r113176495 --- Diff: src/main/java/org/apache/commons/compress/compressors/lzma/LZMACompressorInputStream.java --- @@ -56,7 +56,7 @@ public LZMACompressorIn

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-25 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r113141961 --- Diff: src/main/java/org/apache/commons/compress/compressors/lzma/LZMACompressorInputStream.java --- @@ -56,7 +56,7 @@ public LZMACompresso

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-25 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r113141080 --- Diff: src/main/java/org/apache/commons/compress/compressors/lzma/LZMACompressorInputStream.java --- @@ -56,7 +56,7 @@ public LZMACompressorInputS

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-23 Thread sebbASF
Github user sebbASF commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/21#discussion_r112838915 --- Diff: src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java --- @@ -,14 +1122,11 @@ public int read() throws IOException {

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-23 Thread kvr000
Github user kvr000 commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/21#discussion_r112838864 --- Diff: src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java --- @@ -,14 +1122,11 @@ public int read() throws IOException {

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-23 Thread kvr000
Github user kvr000 commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/21#discussion_r112838711 --- Diff: src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java --- @@ -1081,16 +1082,26 @@ private boolean startsWithLocalFileHeader(

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-23 Thread bodewig
Github user bodewig commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/21#discussion_r112838584 --- Diff: src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java --- @@ -,14 +1122,11 @@ public int read() throws IOException {

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-23 Thread sebbASF
Github user sebbASF commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/21#discussion_r112838372 --- Diff: src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java --- @@ -,14 +1122,11 @@ public int read() throws IOException {

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-23 Thread sebbASF
Github user sebbASF commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/21#discussion_r112838358 --- Diff: src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java --- @@ -1081,16 +1082,26 @@ private boolean startsWithLocalFileHeader

[GitHub] commons-compress pull request #21: COMPRESS-388: Fix concurrent reads perfor...

2017-04-22 Thread kvr000
GitHub user kvr000 opened a pull request: https://github.com/apache/commons-compress/pull/21 COMPRESS-388: Fix concurrent reads performance Ticket: https://issues.apache.org/jira/browse/COMPRESS-388 Concurrent reads on the ZipFile archive is terribly slow on multiprocessor

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-22 Thread bodewig
Github user bodewig commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r112810573 --- Diff: src/test/java/org/apache/commons/compress/compressors/DetectCompressorTestCase.java --- @@ -167,6 +171,26 @@ private String detect(String

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-22 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-compress/pull/20#discussion_r112810239 --- Diff: src/test/java/org/apache/commons/compress/compressors/DetectCompressorTestCase.java --- @@ -167,6 +171,26 @@ private String detect(String t

[GitHub] commons-compress pull request #19: COMPRESS-387

2017-04-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/19 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #18: COMPRESS-385

2017-04-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/18 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #20: COMPRESS-382

2017-04-14 Thread tballison
GitHub user tballison opened a pull request: https://github.com/apache/commons-compress/pull/20 COMPRESS-382 first draft of preventing OOM in LZMA You can merge this pull request into a Git repository by running: $ git pull https://github.com/tballison/commons-compress COMPRESS

[GitHub] commons-compress pull request #19: COMPRESS-387

2017-04-14 Thread tballison
GitHub user tballison opened a pull request: https://github.com/apache/commons-compress/pull/19 COMPRESS-387 Fix specification of ARCDIR to allow spaces in path in Windows You can merge this pull request into a Git repository by running: $ git pull https://github.com/tballison/

[GitHub] commons-compress pull request #18: COMPRESS-385

2017-04-14 Thread tballison
GitHub user tballison opened a pull request: https://github.com/apache/commons-compress/pull/18 COMPRESS-385 add detect() to CompressorStreamFactory You can merge this pull request into a Git repository by running: $ git pull https://github.com/tballison/commons-compress COMPRE

[GitHub] commons-compress pull request #17: CPIO crc overflow resolved for large file...

2017-03-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/17 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #17: CPIO crc overflow resolved for large file...

2017-03-29 Thread dcollin
GitHub user dcollin opened a pull request: https://github.com/apache/commons-compress/pull/17 CPIO crc overflow resolved for large files When unpacking a CPIO file containing a large file the crc check will overflow and throw an IOException("CRC Error..."). Did not find

[GitHub] commons-compress pull request #13: BZip2: Use BitInputStream

2017-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/13 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #16: Don't duplicate count() functionality.

2017-02-04 Thread thomasmey
Github user thomasmey closed the pull request at: https://github.com/apache/commons-compress/pull/16 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #16: Don't duplicate count() functionality.

2017-01-27 Thread thomasmey
GitHub user thomasmey opened a pull request: https://github.com/apache/commons-compress/pull/16 Don't duplicate count() functionality. You can merge this pull request into a Git repository by running: $ git pull https://github.com/thomasmey/commons-compress work/misc Alternat

[GitHub] commons-compress pull request #14: BlockSort: correct spelling

2017-01-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/14 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #15: Some fixes for GZipCompressorInputStream

2017-01-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/15 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #15: Some fixes for GZipCompressorInputStream

2017-01-19 Thread thomasmey
GitHub user thomasmey opened a pull request: https://github.com/apache/commons-compress/pull/15 Some fixes for GZipCompressorInputStream You can merge this pull request into a Git repository by running: $ git pull https://github.com/thomasmey/commons-compress gzip-compressor-

[GitHub] commons-compress pull request #14: BlockSort: correct spelling

2017-01-19 Thread thomasmey
GitHub user thomasmey opened a pull request: https://github.com/apache/commons-compress/pull/14 BlockSort: correct spelling You can merge this pull request into a Git repository by running: $ git pull https://github.com/thomasmey/commons-compress blocksort-spelling Alternativ

[GitHub] commons-compress pull request #:

2017-01-17 Thread bodewig
Github user bodewig commented on the pull request: https://github.com/apache/commons-compress/commit/56e82da90f1064c23dd630cf0066231567da3ed6#commitcomment-20502548 In src/main/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorInputStream.java: In src/main/java/o

[GitHub] commons-compress pull request #:

2017-01-17 Thread bastik
Github user bastik commented on the pull request: https://github.com/apache/commons-compress/commit/56e82da90f1064c23dd630cf0066231567da3ed6#commitcomment-20501836 In src/main/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorInputStream.java: In src/main/java/or

[GitHub] commons-compress pull request #:

2017-01-16 Thread bodewig
Github user bodewig commented on the pull request: https://github.com/apache/commons-compress/commit/56e82da90f1064c23dd630cf0066231567da3ed6#commitcomment-20498632 In src/main/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorInputStream.java: In src/main/java/o

[GitHub] commons-compress pull request #:

2017-01-16 Thread bastik
Github user bastik commented on the pull request: https://github.com/apache/commons-compress/commit/56e82da90f1064c23dd630cf0066231567da3ed6#commitcomment-20495058 In src/main/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorInputStream.java: In src/main/java/or

[GitHub] commons-compress pull request #13: BZip2: Use BitInputStream

2017-01-15 Thread thomasmey
GitHub user thomasmey opened a pull request: https://github.com/apache/commons-compress/pull/13 BZip2: Use BitInputStream And some minor spelling errors I stumbled upon. You can merge this pull request into a Git repository by running: $ git pull https://github.com/thomasmey/co

[GitHub] commons-compress pull request #12: COMPRESS-375 Allow the deferred creation ...

2016-12-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/12 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request #12: COMPRESS-375 Allow the differed creation ...

2016-12-03 Thread plamentotev
GitHub user plamentotev opened a pull request: https://github.com/apache/commons-compress/pull/12 COMPRESS-375 Allow the differed creation of `ZipArchiveEntry` for parallel zips This is my attempt to implement the changes suggested in [COMPRESS-375](https://issues.apache.org/jira/b

[GitHub] commons-compress pull request #:

2016-11-16 Thread PascalSchumacher
Github user PascalSchumacher commented on the pull request: https://github.com/apache/commons-compress/commit/7c93eb18bad8612fe7cd4e6d27bd413b1fafc978#commitcomment-19849282 In README.md: In README.md on line 104: done: https://github.com/apache/commons-compress/commit/14441db

[GitHub] commons-compress pull request #:

2016-11-16 Thread PascalSchumacher
Github user PascalSchumacher commented on the pull request: https://github.com/apache/commons-compress/commit/7c93eb18bad8612fe7cd4e6d27bd413b1fafc978#commitcomment-19842390 In README.md: In README.md on line 104: Looks like some replacing did not work and now the link is dead.

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread rpreissel
Github user rpreissel commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197507339 This is a memory issue and from my point of view there is no simple way to test it with an unit test. I have tried to write an unit test and used Man

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/11 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread rpreissel
GitHub user rpreissel opened a pull request: https://github.com/apache/commons-compress/pull/11 Bugfix for COMPRESS-343 The class ...sevenz.Coders.DeflateDecoder does not close (end()) the Deflater and Inflater. This can lead to native memory issues: see https://bugs.openjdk.java.n

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread garydgregory
Github user garydgregory commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197425393 Hi, Thank you for the patch. Do you have a unit test that show the problem and it being fixed by the patch? Gary --- If your pr

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread bodewig
Github user bodewig commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197788653 I had naively assumed `InflaterInputStream#close` would free up the resources. Come to think of it, if we pass in the `Inflater` ourselves it makes sense that w

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-18 Thread rpreissel
Github user rpreissel commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197554132 I added tests with PowerMock/Mockito. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-18 Thread garydgregory
Github user garydgregory commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197518791 Yes, one of: - Use Mockito or Powermock to assert that the end() method has been called (best) - add an isClosed method somewhere that sets an

[GitHub] commons-compress pull request: Zip64Mode.Always now impacts centra...

2016-03-05 Thread bodewig
Github user bodewig commented on the pull request: https://github.com/apache/commons-compress/pull/10#issuecomment-192735255 Thanks a lot Matt, this makes things quite a bit more consistent. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] commons-compress pull request: Zip64Mode.Always now impacts centra...

2016-03-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/10 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] commons-compress pull request: Zip64Mode.Always now impacts centra...

2016-02-29 Thread weirdkid
GitHub user weirdkid opened a pull request: https://github.com/apache/commons-compress/pull/10 Zip64Mode.Always now impacts central file header same as local file These changes bring the file header entries in the central directory into alignment with the local file headers when zip

[GitHub] commons-compress pull request: Allow other streams supporting rand...

2016-02-27 Thread rchodava
Github user rchodava closed the pull request at: https://github.com/apache/commons-compress/pull/9 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featu

[GitHub] commons-compress pull request: Allow other streams supporting rand...

2016-02-27 Thread rchodava
Github user rchodava closed the pull request at: https://github.com/apache/commons-compress/pull/9 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featu

[GitHub] commons-compress pull request: Allow other streams supporting rand...

2016-02-27 Thread rchodava
GitHub user rchodava reopened a pull request: https://github.com/apache/commons-compress/pull/9 Allow other streams supporting random access to be used by ZipFile, instead of just RandomAccessFile You can merge this pull request into a Git repository by running: $ git pull ht

[GitHub] commons-compress pull request: Allow other streams supporting rand...

2016-02-27 Thread rchodava
GitHub user rchodava opened a pull request: https://github.com/apache/commons-compress/pull/9 Allow other streams supporting random access to be used by ZipFile, instead of just RandomAccessFile You can merge this pull request into a Git repository by running: $ git pull http

[GitHub] commons-compress pull request: COMPRESS-320: add the ability to qu...

2016-02-24 Thread dweiss
Github user dweiss commented on the pull request: https://github.com/apache/commons-compress/pull/8#issuecomment-188401981 You're welcome, thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] commons-compress pull request: COMPRESS-320: add the ability to qu...

2016-02-24 Thread bodewig
Github user bodewig commented on the pull request: https://github.com/apache/commons-compress/pull/8#issuecomment-188288771 merged, many thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not ha

[GitHub] commons-compress pull request: COMPRESS-320: add the ability to qu...

2016-02-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/8 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] commons-compress pull request: COMPRESS-320: add the ability to qu...

2016-02-23 Thread dweiss
GitHub user dweiss opened a pull request: https://github.com/apache/commons-compress/pull/8 COMPRESS-320: add the ability to quickly "skip" entries while scanning 7z archive As described in JIRA: https://issues.apache.org/jira/browse/COMPRESS-320 You can merge this pull request

[GitHub] commons-compress pull request: COMPRESS-333: adds buffering on top...

2016-02-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/7 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

  1   2   >