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

refactor(launcher): refactor client APK installation from launcher #2531

Merged
merged 2 commits into from
Nov 29, 2024

Conversation

E1int
Copy link
Contributor

@E1int E1int commented Nov 26, 2024

This PR makes the launcher use the ADB commands from alvr_adb and should fix the version downgrade error that could occur when installing the client APK. The download location of the local ADB has been changed to be in the release folder.

Copy link
Member

@zmerp zmerp left a comment

Choose a reason for hiding this comment

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

Beside the comments below, I think it would be nice to also add automatic launching of the client once installed. So you can rename install_apk() to install_and _launch_apk().

alvr/launcher/src/actions.rs Outdated Show resolved Hide resolved
alvr/launcher/src/actions.rs Outdated Show resolved Hide resolved
alvr/launcher/src/actions.rs Show resolved Hide resolved
@The-personified-devil The-personified-devil merged commit 587bdc3 into alvr-org:master Nov 29, 2024
8 checks passed
zmerp pushed a commit that referenced this pull request Dec 7, 2024
…2531)

* Refactor client APK installation from launcher

* Apply requested changes
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