dtrx-py / dtrx

Do The Right Extraction
GNU General Public License v3.0
230 stars 10 forks source link

Fixes for 7z #49

Closed ChrisJefferson closed 5 months ago

ChrisJefferson commented 1 year ago

This fixes #48 .

I removed another use of "string.rindex" at the same time -- I can't actually make the code execute, but it might be I just couldn't figure out how to do it, and I certainly haven't made the code worse by removing an invalid function. Also added text.

I actually changed how 7z reads files, as once I fixed the string.rindex code, the result still didn't work. Added a test.

ChrisJefferson commented 9 months ago

Rebased on top of #54 to fix CI, just so I can see if tests pass

DeflateAwning commented 5 months ago

I think I may have encountered one of the cases that this fixes. Any chance this could get merged?