Add a dependency on dissect.fve for the full extra

[pyrco]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.82%. Comparing base (1a2c8fe) to head (503b1cb).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #949      +/-   ##
==========================================
+ Coverage   77.50%   77.82%   +0.31%     
==========================================
  Files         323      323              
  Lines       27647    27647              
==========================================
+ Hits        21429    21517      +88     
+ Misses       6218     6130      -88     

Flag	Coverage Δ
unittests	77.82% <ø> (+0.31%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Zawadidone]

@pyrco does this platform_python_implementation != 'PyPy' mean that dissect.fve does not work with PyPy, as part of dissect.target?

[Miauwkeru]

@pyrco does this platform_python_implementation != 'PyPy' mean that dissect.fve does not work with PyPy, as part of dissect.target?

it does an or operation in this dependency fix, so first it evaluates whether it isn't windows, then it evaluates if it isn't pypy.

So it would come through on linux machines with pypy.

The check is like this to avoid it installing on windows that uses PyPy, as it gave issues. The logic is inverted however to let it install on other systems

[Zawadidone]

Okay thanks for the explanation.