On Mon, 17 Mar 2025, Roger Pau Monne wrote:
> Instead use env to find the location of expect.
>
> Additionally do not use the -f flag, as it's only meaningful when passing
> arguments on the command line, which we never do for console.exp. From the
> expect 5.45.4 man page:
>
> > The -f flag prefaces a file from which to read commands from. The flag
> > itself is optional as it is only useful when using the #! notation (see
> > above), so that other arguments may be supplied on the command line.
>
> Signed-off-by: Roger Pau Monné <roger....@citrix.com>
Do you have a link to a successful Gitlab and Cirrus CI?
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
> ---
> automation/scripts/console.exp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/automation/scripts/console.exp b/automation/scripts/console.exp
> index 310543c33e5e..31ce97b91b63 100755
> --- a/automation/scripts/console.exp
> +++ b/automation/scripts/console.exp
> @@ -1,4 +1,4 @@
> -#!/usr/bin/expect -f
> +#!/usr/bin/env expect
>
> if {[info exists env(TEST_TIMEOUT_OVERRIDE)]} {
> set timeout $env(TEST_TIMEOUT_OVERRIDE)
> --
> 2.48.1
>
