Message ID | 20180503122222.2996-4-pvorel@suse.cz |
---|---|
State | Accepted |
Delegated to: | Petr Vorel |
Headers | show |
Series | net: Cleanup variables in TST_PARSE_ARGS functions for legacy API | expand |
diff --git a/testcases/network/virt/virt_lib.sh b/testcases/network/virt/virt_lib.sh index 0f75bb030..1fd02ac9b 100644 --- a/testcases/network/virt/virt_lib.sh +++ b/testcases/network/virt/virt_lib.sh @@ -31,7 +31,7 @@ virt_lib_parse_args() { - case "$opt" in + case "$1" in h) echo "Usage:" echo "h help" @@ -40,8 +40,8 @@ virt_lib_parse_args() echo "6 run over IPv6" exit 0 ;; - i) start_id=$OPTARG ;; - d) vxlan_dst_addr=$OPTARG ;; + i) start_id=$2 ;; + d) vxlan_dst_addr=$2 ;; *) tst_brkm TBROK "unknown option: $opt" ;;
as it's now supported also for legacy API. Major problem was with using global $opt instead of $1 (dependency on how the library function implemented), but avoiding using $OPTARG directly is also preferable. Signed-off-by: Petr Vorel <pvorel@suse.cz> --- testcases/network/virt/virt_lib.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)