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: fix decompress issue with rar files which include CJK filenames in it #759

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

ttys3
Copy link
Contributor

@ttys3 ttys3 commented Nov 30, 2024

update unrar crate to fix decompress issue with rar files which include CJK filenames in it

prev we fixed the CJK filename issue for the archive filename itself

this time, it is the filename in the archive.

that's to say, if foo.rar contains ❤️.txt in it, the decompression will fail

we need wait unrar crate to release a new version

@ttys3 ttys3 changed the title fix: update unrar crate to fix deompress issue with rar files which i… fix: fix decompress issue with rar files which include CJK filenames in it Nov 30, 2024
@marcospb19
Copy link
Member

marcospb19 commented Dec 2, 2024

Nice PR.

Yeah, we need to wait for their release otherwise this can block a new version of Ouch.

@ttys3 ttys3 marked this pull request as draft December 2, 2024 08:28
@ttys3
Copy link
Contributor Author

ttys3 commented Dec 2, 2024

OK, I changed the PR to WIP status

@ttys3 ttys3 force-pushed the fix/rar-decompress branch from f4d42be to 9c70ebf Compare December 5, 2024 11:06
@ttys3 ttys3 marked this pull request as ready for review December 5, 2024 11:06
@ttys3 ttys3 force-pushed the fix/rar-decompress branch from 9c70ebf to 8491a88 Compare December 5, 2024 11:06
Copy link
Member

@marcospb19 marcospb19 left a comment

Choose a reason for hiding this comment

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

looks like you also updated all deps, I guess it's a good thing tho

@marcospb19 marcospb19 merged commit 493213e into ouch-org:main Dec 6, 2024
15 checks passed
@marcospb19
Copy link
Member

marcospb19 commented Dec 6, 2024

ty, I see you created another PRs, sorry, I might take some time to take a proper look at them, busy weeks, but I'll catch up when I can

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