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(unrar): extract_to should set parent path #50

Closed
wants to merge 1 commit into from

Conversation

deanrock
Copy link

@deanrock deanrock commented Apr 22, 2024

It seems that just specifying the file does not work with absolute paths, e.g. /tmp/folder/filename.txt.

After doing some testing, it seems that providing parent folder as path and complete path as file is the only thing that consistently works.

I've also added tests, to ensure that both extraction to relative and absolute paths work as expected.

@muja muja closed this Aug 31, 2024
@muja muja reopened this Aug 31, 2024
@muja
Copy link
Owner

muja commented Aug 31, 2024

This is unnecessary and has already been fixed by the new UnrarDLL version. See #34 for more details

@muja muja closed this Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants