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

Give unzip priority over p7zip #279

Closed
wants to merge 1 commit into from
Closed

Give unzip priority over p7zip #279

wants to merge 1 commit into from

Conversation

vkareh
Copy link
Member

@vkareh vkareh commented Jul 8, 2019

At the moment p7zip does not process OEM encoding in file names (https://sourceforge.net/p/p7zip/bugs/187/) so instead we prioritize unzip.

Fixes #201
Fixes #5

At the moment p7zip does not process OEM encoding in file names, so
instead we prioritize unzip.
@vkareh vkareh requested a review from a team July 8, 2019 23:29
@sc0w
Copy link
Member

sc0w commented Jul 9, 2019

it doesn't work here.

anyway, unzip only supports "read only", so, I don't like it, it must work maybe with a gsettings key

(EDIT: it works in read/write mode, I don't know how it works, but it works)

and, we have the ability to use unar to open zip files since c587ae1 .

I noticed using unar fixes #278 , and probably more issues.

@vkareh
Copy link
Member Author

vkareh commented Jul 9, 2019

@sc0w that's a good point, but I think this should work automatically. For a user to have to change gsettings to open a file is not good UX, maybe we can prioritize unar instead of unzip? Why do we need the gsettings instead of just making it automatic?

@sc0w
Copy link
Member

sc0w commented Jul 9, 2019

@vkareh because unar open zip files with "read only" mode.

I noticed this PR fixes #201 but it doesn't fix the name of files in the other issues you mentioned in #279 (comment)

and zip files are opened locally in "read / write" mode, this is a good point.

@vkareh
Copy link
Member Author

vkareh commented Jul 9, 2019

unzip only supports "read only"

and

unar open zip files with "read only" mode

and

zip files are opened locally in "read / write" mode

Those three statements confuse me a bit, which ones open in read-only and which ones open in read/write mode?

This fixes the file name issues for me. I wonder what's different between our configurations. I'll play around with settings to see if I find anything.

@sc0w
Copy link
Member

sc0w commented Jul 9, 2019

Those three statements confuse me a bit, which ones open in read-only and which ones open in read/write mode?

All are opened in read/write mode, I was wrong, I edited that before. I don't know how it works, but it works.

now I agree with this PR because fixes the issue of password, but I don't understand why the names of files works for you, and it doesn't work here.

awaiting the tests of others...

@sc0w
Copy link
Member

sc0w commented Jul 9, 2019

ok, seeing the code I noticed it uses unzip to show the files, and the command zip to compress

Copy link
Member

@sc0w sc0w left a comment

Choose a reason for hiding this comment

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

This PR fixes #201 but it doesn't fix the name of files for me in the other issues you mentioned in #279 (comment) , so, it only needs to clarify and confirm which issues fixes this PR.

And it works in read/write mode, so, I am happy with it.

@sc0w sc0w requested a review from a team July 9, 2019 02:03
@vkareh
Copy link
Member Author

vkareh commented Jul 9, 2019

Thanks for the clarification :)

I still haven't found why it may not work for you for file names... I'll keep trying different things on my side to reproduce the issue

@vkareh
Copy link
Member Author

vkareh commented Jul 9, 2019

@sc0w, I've edited the list of issues this will fix. All other issues are already closed, and the only other one (#102) seems to display the wrong character set with either p7z, unar, or unzip. So I removed it from this list. We're only left with #5 (feature request to prioritize unzip) and #201 (password encoding).

@sc0w
Copy link
Member

sc0w commented Jul 11, 2019

I noticed one problem with this PR with .zip files compressed with password by 7z

how to reproduce:

without this PR, select some files in caja, right click, compress in zip, other options, set password, and create. (it is created by 7z).

install this PR, and try to open, it doesn't work, seems it isn't supported by unzip.

file sample, the pass is 123:

pass123.zip

skipping: engrampa_1.20.2-1_amd64.buildinfo need PK compat. v5.1 (can do v4.6)

Copy link
Member

@raveit65 raveit65 left a comment

Choose a reason for hiding this comment

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

Travis checks are fine.
I don't care about which backend is used.

@sc0w sc0w self-requested a review July 17, 2019 12:56
Copy link
Member

@sc0w sc0w left a comment

Choose a reason for hiding this comment

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

I mentioned the reasons in my previous comment

We will have problems with some zip files compressed with passwods with this PR, maybe we need gsettings key.

@raveit65
Copy link
Member

raveit65 commented Oct 9, 2019

What happens with that PR?
Is someone working on it or can we close it?

@raveit65
Copy link
Member

@vkareh @sc0w
Any news?

@vkareh
Copy link
Member Author

vkareh commented Jun 5, 2020

There hasn't been consensus in a year. Closing for now.

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.

Cannot decrypt zip with "№" symbol in password Prefer to open zip files with unzip instead of 7z
3 participants