<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 9, 2021 at 10:01 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Petr Vorel <<a href="mailto:petr.vorel@gmail.com" target="_blank">petr.vorel@gmail.com</a>><br>
<br>
in tst_ipaddr_un(). This is required to fix problem on dash, which<br>
(unlike bash and busybox ash implementation) takes value of previously<br>
defined variable, i.e.  guarding with local does not work:<br>
<br>
$ foo=ee; bar() { local foo; echo "foo: '$foo'"; }; bar<br>
foo: 'ee'<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I'm thinking maybe we should avoid defining the same name</div><div class="gmail_default" style="font-size:small">variables like that. But anyway, your fix should be worked.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
It requires declare it as empty:<br>
$ foo=ee; bar() { local foo=; echo "foo: '$foo'"; }; bar<br>
foo: ''<br>
<br>
This problem seems to be on various dash versions from 0.5.8 to<br>
0.5.11+git20200708+dd9ef66-5 (on Debian oldstable, stable, testing).<br>
It'd require to set all variables, but now fix just prefix, which is<br>
used in build.sh and it's quite common name.<br>
<br>
Signed-off-by: Petr Vorel <<a href="mailto:petr.vorel@gmail.com" target="_blank">petr.vorel@gmail.com</a>><br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
 testcases/lib/tst_net.sh | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/testcases/lib/tst_net.sh b/testcases/lib/tst_net.sh<br>
index db1db4790..52975f37b 100644<br>
--- a/testcases/lib/tst_net.sh<br>
+++ b/testcases/lib/tst_net.sh<br>
@@ -408,7 +408,7 @@ tst_ipaddr_un()<br>
        local max_net_id=$default_max<br>
        local min_net_id=0<br>
<br>
-       local counter host_id host_range is_counter max_host_id min_host_id net_id prefix tmp type<br>
+       local counter host_id host_range is_counter max_host_id min_host_id net_id prefix= tmp type<br>
<br>
        local OPTIND<br>
        while getopts "c:h:n:p" opt; do<br>
-- <br>
2.26.2<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>