[WIP] backend: Fix sh_init stop function pointer

[pevik]

s/ssh_stop/sh_stop/

Obvious typo.

Reported-by: Andrea Cervesato [email protected]
Signed-off-by: Petr Vorel [email protected]

[pevik]

Also backport one commit from https://github.com/metan-ucw/runltp-ng/pull/46/commits to fix CI.

[pevik]

Ah, I overlooked sh_stop does not exists, it was ssh_stop.

[metan-ucw]

Looking at the code it may not be mistake at all.

Notice that both ssh_start and sh_start does set self->{$pid} so stopping a shell backend by running exit command and then waitpid on the $pid looks like it may work actually. Unfortunately I no longer remember if this was a design or if it works by accident.

[pevik]

OK, up to you.

Could you please cherry-pick and merge the first commit which fixes CI? (in case somebody tries to send PR it'd be good CI is working well).

[cfconrad]

Only that this line https://github.com/metan-ucw/runltp-ng/blob/master/backend.pm#L559 will produce a
Use of uninitialized value $user in string eq at warning.

I would be for creating the sh_stop() and simply call the exit.