-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add password options for 7z, zip, rar and arj #51
Conversation
Thanks. This looks like a good direction! One useful thing to add would be tests for all the encrypted formats we want to support (the way tests are implemented isn't really documented, but it's (hopefully) not too hard to understand. The test will (I imagine) look a lot like a current test (could take the existing files and just add encryption?) -- the pass In terms of python 2, my preference would be to support it if it's not that difficult -- as dtrx is an "easy to use" end-user program, there are a bunch of people running old OSes which are still on Python 2, who use dtrx (I know a couple personally), who I wouldn't want to have to teach how to install Python 3. Of course, if something turns out to be difficult to implement in Python 2, that would be a reason to abandon support. |
Thank you Chris for your quick response. I love dtrx for my daily work and it is also a great tool for automation. With the password option added I can use it as a full replacement for patool. That would be so good. For this PR it is a no brainer to support py v2. I only noticed the v2 support after writing the code. Of course I will write tests as well. |
Okay, I've added the password option to unar and arj as well. And added some tests. I would appreciate your feedback if something is missing. |
Hi, just to say sorry I've been slow looking at this, life has gotten busy, but it is on my list to get to. |
This would be great to get merged! |
Anything I can do to get this merged, @ChrisJefferson? |
Sorry for the delay, I tested and this all looks good. |
Just curious, what's the plan for a new release? I'd like to play around with this new feature, but if there's a planned release then I'll just wait for that so I can simultaneously update the debian package. @noahp |
Closes #50
Since I want to use dtrx to extract archives in non-interactive mode, even if they are password protected, I started to write a patch to enable dtrx to do this. This PR is currently a draft because
If this draft seems good, I'd like to add password support for the other extractors as well (for those libraries that support encrypted archives).
Regarding f-strings: Shouldn't dtrx drop support for Python 2?
I'm happy to hear your comments!