Inline... On 6/26/12 9:43 AM, openwrt-comm...@openwrt.org wrote: > Author: jow > Date: 2012-06-26 17:43:09 +0200 (Tue, 26 Jun 2012) > New Revision: 32505 > > Modified: > trunk/package/broadcom-wl/files/etc/hotplug.d/net/20-broadcom_wds > Log: > [package] broadcom-wl: fix WDS hotplug for multiple networks > > Modified: trunk/package/broadcom-wl/files/etc/hotplug.d/net/20-broadcom_wds > =================================================================== > --- trunk/package/broadcom-wl/files/etc/hotplug.d/net/20-broadcom_wds > 2012-06-26 15:40:03 UTC (rev 32504) > +++ trunk/package/broadcom-wl/files/etc/hotplug.d/net/20-broadcom_wds > 2012-06-26 15:43:09 UTC (rev 32505) > @@ -23,7 +23,9 @@ > include /lib/network > scan_interfaces > > - setup_interface "$iface" "$network" > + for network in $network; do > + setup_interface "$iface" "$network" > + done
Has this been tested? I'm not even sure that it's guaranteed to work... you're counting on the order or dollar-expansion, etc. It might just be clearer to use a second variable name for the loop iterator (and that leaves the original contents of 'network' intact). In an interpreted language it might work, but for someone who looks at compiled code all day that just looks terribly wrong. :-( > > config_get encryption "$cfg" encryption > config_get key "$cfg" key > _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel
