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
Android app start fix #477
Conversation
Don't wait for emulator to boot.
@@ -2110,6 +2127,7 @@ int main (const int argc, const char* argv[]) { | |||
auto cnt = 0; | |||
|
|||
AndroidCliState androidState; | |||
androidState.targetPlatform = targetPlatform; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you may need to do the same for ssc run
command
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can move the androidState
populating to the run
function?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thanks 😅
maybe we can move the androidState populating to the run function?
Let's slate this for the next release so we can test it a bit more?
Just to confirm, does run
get called by build
?
It might be better to have a platformInit()
that in turn calls platform specific setup.
AndroidState should eventually contain everything necessary for build, currently it only covers run.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to confirm, does run get called by build?
yes
cli: android: fail immediately if adb fails to install to device
Don't wait for emulator to boot.