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

Feature/supporting new compression formats #747

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fatihergin
Copy link
Contributor

@fatihergin fatihergin commented Sep 10, 2023

Highlights

  • The PR only covers supporting new compression formats .7z, .tar.gz and .tar.xz.
  • Decompressing not handled in this PR.
  • Only .7z supports reading password protected files out of .7z, .tar.gz and .tar.xz.
  • Creating password protected files is not supported by the new formats. Existing password protected .zip solution still works as it is.
  • New minimum Android SDK is now 26(as it's decided) because of the Apache Commons Compress's internal Java 8 dependencies.
  • 2 TODOs exist for the new strings. I didn't add them to the language files without finalizing the keys & translations
    https://github.com/SimpleMobileTools/Simple-File-Manager/pull/747/files#diff-5e01f7d37a66e4ca03deefc205d8e7008661cdd0284a05aaba1858e6b7bf9103R38

Please find below resources for decompressing samples of apache commons compress used in this PR

https://commons.apache.org/proper/commons-compress/examples.html
https://itsallbinary.com/apache-commons-compress-sevenzip-unsevenzip-examples-7z/
https://itsallbinary.com/apache-commons-compress-tar-gzip-bzip2-xz-snappy-deflate-examples-un-gzip-untar-examples/

@fatihergin fatihergin force-pushed the feature/supporting-new-compression-formats branch from 1455a26 to aa247e9 Compare September 26, 2023 20:00
@fatihergin fatihergin force-pushed the feature/supporting-new-compression-formats branch from de93884 to edd3f93 Compare October 6, 2023 18:51
@fatihergin fatihergin marked this pull request as ready for review October 6, 2023 19:26
@inson1
Copy link

inson1 commented Feb 15, 2024

@fatihergin SMT apps were sold to ad company. Fork is here https://github.com/FossifyOrg
Could you do PR there?

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.

None yet

2 participants