[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH 7/8] automation/console.exp: do not assume expect is always at /usr/bin/
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.pau@xxxxxxxxxx> Do you have a link to a successful Gitlab and Cirrus CI? Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx> > --- > 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 >
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |