这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@dorpvom
Copy link
Member

@dorpvom dorpvom commented Jul 30, 2018

As of now debian packages are handled by the generic_carver. This results in the typical false positive .xz and .zlib archives.

@dorpvom dorpvom self-assigned this Jul 30, 2018
@dorpvom dorpvom requested a review from weidenba July 30, 2018 12:59
@codecov-io
Copy link

codecov-io commented Jul 30, 2018

Codecov Report

Merging #151 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #151   +/-   ##
=======================================
  Coverage   95.71%   95.71%           
=======================================
  Files         302      302           
  Lines       14959    14959           
=======================================
  Hits        14318    14318           
  Misses        641      641

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0076fc2...dd750c4. Read the comment docs.



def unpack_function(file_path, tmp_dir):
if not program_is_callable("fakeroot"):
Copy link
Contributor

Choose a reason for hiding this comment

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

new policy says: Do not check the installation in production code. Pytest should tell us anyhow

from test.unit.unpacker.test_unpacker import TestUnpackerBase


TEST_DATA_DIR = os.path.join(os.path.dirname(os.path.abspath(__file__)), 'data')
Copy link
Contributor

Choose a reason for hiding this comment

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

use pathlib if possible

@weidenba weidenba merged commit 403b2c2 into master Aug 1, 2018
@weidenba weidenba deleted the debian-package-unpacker branch August 1, 2018 09:33
weidenba added a commit that referenced this pull request Oct 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants