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

ZIP - can not unzip on external sd #478

Closed
dukelc opened this Issue Sep 3, 2016 · 6 comments

Comments

2 participants
@dukelc

dukelc commented Sep 3, 2016

Can not unzip on external sd. (exfat)
On internal the same file - unziping as it should.

Phone galaxy s4 i9505
Android 6.0.1

Logs should be in bug repport.

@d4rken

This comment has been minimized.

Owner

d4rken commented Sep 3, 2016

Can you record a debug log?

When you turn on debug mode (i.e. during a debug run setting to debuglevel 2), in Overview, tap the entry for the external storage, what access mode does it display (bottom)?

@dukelc

This comment has been minimized.

dukelc commented Sep 3, 2016

@d4rken

This comment has been minimized.

Owner

d4rken commented Sep 3, 2016

Long press the settings entry.

See https://github.com/d4rken/sdmaid-public/wiki/Reporting-a-bug for more info on that stuff.

Thanks for the log will check it later.

@d4rken

This comment has been minimized.

Owner

d4rken commented Sep 4, 2016

Good catch.
Right... Just looked at the code again and that is just not implemented...
Normal sdcard can use java based OutputStream.
External sdcard access needs either unzipping via root or using an outputstream that we get from Androids Storage Access Framework....

😦 will have to look into that.

@d4rken d4rken added this to the Next Tasks milestone Sep 4, 2016

@d4rken

This comment has been minimized.

Owner

d4rken commented Sep 4, 2016

Also saw that SD Maid does not log unzipping errors, added that for the next update.

@d4rken d4rken modified the milestones: v4.3.3, Next Tasks Sep 17, 2016

@d4rken

This comment has been minimized.

Owner

d4rken commented Sep 17, 2016

Should work in the next version, a bit more work than I thought because if the user skipped the secondary storage setup and tries to extract something that requires it we have to abort and ask the user to complete the setup.

@d4rken d4rken closed this Sep 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment