This adapter is only ever run by the Radicle CI broker, and that can
now limit CI run duration itself. Thus having this adapter do it too
is pointless duplication.
Further, this removes the run time dependency on an external "timeout"
command, which turns out not to be portable: it isn't available on
Macs, by default.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
They're in the build log too, and come from the trigger message, but
it's helpful for debugging to log them clearly.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
The CI broker has dropped the RunResult::Error variant, so drop any
use of that. The Request::version field has been added, so deal with
that.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
If any command run by CI for a repository fails, set the run to failed
in the run info builder.
Also, we were hiding that failure with later errors, if, say, updating
a log file or removing temporary files failed, so don't do that.
In the old engine it's hard to follow how various errors are handled.
This new engine should make it clearer.
This isn't entirely a pure refactoring. It subtly changed how the
external behavior is, to fix problems.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
Lars Wirzenius