On Thu, Feb 5, 2026 at 1:50 AM Ihar Hrachyshka <[email protected]> wrote:
> `which` may be not installed in test environment. If so, no installed > commands were recognized as available for tests. > > Signed-off-by: Ihar Hrachyshka <[email protected]> > --- > tests/atlocal.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/atlocal.in b/tests/atlocal.in > index 428088c41..477d56a0f 100644 > --- a/tests/atlocal.in > +++ b/tests/atlocal.in > @@ -140,7 +140,7 @@ find_l7_lib sctp > # HAVE_COMMAND="yes", otherwise HAVE_COMMAND="no". > find_command() > { > - which $1 > /dev/null 2>&1 > + command -v "$1" > /dev/null 2>&1 > status=$? > var=HAVE_`echo "$1" | tr '-' '_' | tr '[a-z]' '[A-Z]'` > if test "$status" = "0"; then > -- > 2.52.0 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Thank you Ihar, shouldn't the same change be done for ovs? Regardless I went ahead, added the missing Fixes tag, merged this into main and backported all the way down to 24.03. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
