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

Add Android App Bundle (.aab) file support. #68

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ArkaprabhaChakraborty
Copy link

Signed-off-by: ArkaprabhaChakraborty <[email protected]>
@ArkaprabhaChakraborty
Copy link
Author

Checked the CI job. My tests have passed but there are others that are breaking:

tests/test_androidappbundle.py::TestExtractAAB::test_can_identify_aab_file 
tests/test_androidappbundle.py::TestExtractAAB::test_can_extract_aab_file 
[gw1] PASSED tests/test_androidappbundle.py::TestExtractAAB::test_can_identify_aab_file 
tests/test_androidappbundle.py::TestExtractAAB::test_extract_aab_nonexistent_file 
[gw1] PASSED tests/test_androidappbundle.py::TestExtractAAB::test_extract_aab_nonexistent_file 
tests/test_archive.py::TestGetExtractorTest::test_7zip_extract_can_extract_to_relative_paths 

Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

Thanks...

  1. Can you find a very tiny bundle for testing? We also need to know where it is coming from and add an .ABOUT file to document its origin if possible, or a note explaining how to create it. Then please amend your commit with this small file
  2. Is an aab a zip? if this is so, you should just add a handler for this as in
    # Archive handlers.
    ... Do not recreate zip extraction. Same with the tests, if this a zip, then just add that file to the zip tests.
  3. Is this code that was generated somehow? We are not against it, but this need to be stated upfront.

Generated code is most often approximate, and non-integrated in the rest of the code like your code here and it requires an inordinate amount of effort from maintainers to review, so we need to know

@ArkaprabhaChakraborty
Copy link
Author

ArkaprabhaChakraborty commented Jan 29, 2025 via email

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.

Add support for Android .AAB app bundles
2 participants