[LTP] [PATCH 2/3] network/tst_net.sh: fix busybox/sysctl in tst_set_sysctl()
Petr Vorel
pvorel@suse.cz
Tue Aug 24 14:11:24 CEST 2021
Hi Alexey,
..
> >> - tst_net_run $safe $rparam "sysctl -q -w $name=$value"
> >> + tst_net_run $safe $rparam "sysctl -q -w" "$name=$value"
> > I wonder if it were a bit clearer if we moves all params to second arg:
> > tst_net_run $safe "sysctl" "$name=$value -q -w"
> OK, but changed to "sysctl" "-q -w $name=$value" because the other
> options should also be placed before the key/value pair:
> Usage: sysctl -p [-enq] [FILE...] / [-enqaw] [KEY[=VALUE]]...
Thanks for addressing it!
Kind regards,
Petr
More information about the ltp
mailing list