Re: [OE-core] [PATCH v2 0/1] Change the serial runner usage

Richard Purdie
2023-03-29T10:53:15Z

On Fri, 2023-03-03 at 10:01 +0100, Louis Rannou wrote:
> The actual serial runner has a different usage compare to the ssh runner. The
> return status is different and failure are not raised as exceptions.
>
> Initially, I wanted to create a new run_serial_socket and modify the old
> run_serial to use the former. And there was a second patch that changed every
> call to run_serial to run_serial_socket. But that is not easy as each test
> can have a different manner to handle the exception.
>
> Therefore, this patch only suggest a new runner and add a comment to deprecate
> the former runner.
>
> Louis Rannou (1):
> oeqa/utils/qemurunner: change the serial runner usage
>
> meta/lib/oeqa/targetcontrol.py | 3 ++
> meta/lib/oeqa/utils/qemurunner.py | 57 +++++++++++++++++++++++++++++++
> 2 files changed, 60 insertions(+)

Sorry about the late reply to this. I think the idea of the new runner
is great and we should improve and clean this up.

As the patch stands, it creates a largely duplicate function with no
users. Experience shows this tends to bitrot quickly.

For that reason I'm reluctant to take just this change without
converting at least some of the existing users to use the new API.

Ideally we'd convert everything but at the very least the new function
should be usable by a decent chunk of the existing code, even if we
have to adjust it to make that work.

Cheers,

Richard