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 Helium backups support #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pierrezurek
Copy link

No description provided.

@nelenkov
Copy link
Owner

What is the Helium specific part about this? It looks you just added a tar library which extracts the tar archive, which you can also do with the tar command.

@pierrezurek
Copy link
Author

Helium seems to use the same tar library instead of the Android Backup Service to pack the tar content inside the generated .ab file. Using android-backup-extractor's unpack function on an Helium .ab file gives an invalid .tar file (the tar command fails to extract it). Using the same library to unpack the Helium .ab seems to work fine.
Note : this new option also works fine with .ab files obtained with the adb backup command, maybe it could simply be called unpack-todir ? and we could just add a note somewhere to indicate Helium backups can be extracted with this option ?

@nelenkov
Copy link
Owner

OK, thanks. So it seems Helium produces broken tar files, not sure if it's a good idea to encourage this :)
Can you post a sample .ab archive and the resulting .tar file so I can look at it?

@pierrezurek
Copy link
Author

com example android bluetoothchat ab
out tar

@pierrezurek
Copy link
Author

I added a .jpg extension to upload them here (right click and save as).

@HoverHell
Copy link

And it turns out this is still a problem.

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.

3 participants