-
Notifications
You must be signed in to change notification settings - Fork 179
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
A better way to accommodate custom adb calls #446
Comments
This issue looks interesting to me. I can work on this issue if granted and this is still a valid issue. Thanks! |
This has not come up a lot recently, so it has not been a priority. What idea do you have in mind for the adb level API? |
Thanks for your attention! It all depends on the requirements we have. e.g.:
If I can have more information about the discussion you had with @winterfroststrom so to learn more about details about requirements and your expectation on the result of this issue, then I can come up with a high-level proposal on API design for your review. Again, thanks for your asking and look forward to making a contribution to this repo. |
I see. So basically, there is no good support for non-blocking long running adb commands today. So the idea for this request is to create a mid-layer between raw process management and |
Understood! Let me study some existing packages such as invoke, python-shell, pexpect and python-rrmngmnt to see if there is anything we can learn from them and then will propose a design document (module with API usage) for your review. Thanks! |
Hi, I made up slides as the proposal for this issue. Please check this link:
Any advice and feedback are appreciated. Sorry for the late response and look forward to your lesson. |
This came up in a discussion with @winterfroststrom
The
AdbProxy
was designed for the most straightforward use cases, likeadb.shell('getprop')
- simple call and return.This does not work well if users want to have more control of the call.
E.g. process the live streaming of the stdout of the call subprocess.
We might want to consider providing a new adb interface for users who want to make long-running calls and have more control of the subprocess.
The text was updated successfully, but these errors were encountered: