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

Replace Command handler with libsu #55

Merged
merged 5 commits into from
Jun 26, 2020

Conversation

Tiefkuehlpizze
Copy link

@Tiefkuehlpizze Tiefkuehlpizze commented Jun 24, 2020

Libsu keeps a shell open in the background. This reduces a huge overhead caused by numerous su calls especially during backup/restore.
It changed a lot. It should be well tested.

/update:
Contains changes from PR #50 "Removed: oab-utils in favour of toybox."
toybox is shipped with Android and offers the needed functionality

Toyboxs is shipped since Android 6, but OEMs could have removed it. On these devices/configuration oandbackupx won't work.

toybox is shipped with Android and offers the needed functionality
Changed: Refactored doBackup method to use new libsu wrapper + minor logic changes
Changed: Check for toybox. It can also be busybox. Renamed the variable to have consistent name
Changed: killPackage, setPermissions, checkToybox use libsu wrapper
Fixed: compressing detects if a target does not exists and aborts without failing to avoid triggering errors
Added: Utility functions to convert iterables to a String (used to convert libsu stdout and stderr to simple strings)

Sorry for the long list of changes. It just happened
@Tiefkuehlpizze
Copy link
Author

Rebased changes onto changes up to release 1.3.0

Changed: Refactored doRestore, restoreUserApk, restoreSystemApk, uninstall, getUsers and getDisabledPackages to use libsu wrapper
@machiav3lli machiav3lli merged commit 6c93036 into NeoApplications:master Jun 26, 2020
@Tiefkuehlpizze Tiefkuehlpizze deleted the libsu branch June 27, 2020 13:19
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