Merge "adb: treat non-successful inc-server runs as failures"

This commit is contained in:
Treehugger Robot 2020-06-03 21:28:53 +00:00 committed by Gerrit Code Review
commit ff502379b7

View file

@ -180,7 +180,7 @@ std::optional<Process> install(const Files& files, const Args& passthrough_args,
int print_fds[2];
if (adb_socketpair(print_fds) != 0) {
if (!silent) {
fprintf(stderr, "Failed to create socket pair for child to print to parent\n");
fprintf(stderr, "adb: failed to create socket pair for child to print to parent\n");
}
return {};
}
@ -207,10 +207,15 @@ std::optional<Process> install(const Files& files, const Args& passthrough_args,
Result result = wait_for_installation(pipe_read_fd);
adb_close(pipe_read_fd);
if (result == Result::Success) {
// adb client exits now but inc-server can continue
serverKiller.release();
if (result != Result::Success) {
if (!silent) {
fprintf(stderr, "adb: install command failed");
}
return {};
}
// adb client exits now but inc-server can continue
serverKiller.release();
return child;
}