Bug 119629 – rcscripts/net.modules.d/iwconfig wont find "${conf} " if not eval'ed

• Status: VERIFIED FIXED
• Product: Gentoo Linux

[Description Mark Dominik Bürkle 2006-01-19 18:12:02 UTC]

I guess that whitespace is not at the place it was intended to be.
Despite of that I'd expect it to work as intended as long as that "eval" is still there. If "eval" is removed, it will most likely stop working, if not changed:

# diff -u /lib/rcscripts/net.modules.d/._cfg0000_iwconfig /tmp/
--- /lib/rcscripts/net.modules.d/._cfg0000_iwconfig     2006-01-20 03:02:20.000000000 +0100
+++ /tmp/._cfg0000_iwconfig     2006-01-20 03:02:14.000000000 +0100
@@ -163,7 +163,7 @@
        # Apply the user configuration
        conf="iwconfig_${ifvar}"
        if [[ -n ${!conf} ]]; then
-               aconf=( "${!conf} ")
+               aconf=( "${!conf}" )
                for conf in "${aconf[@]}" ; do
                        if ! eval iwconfig "${iface}" "${conf}" ; then
                                ewarn "${iface} does not support the following configuration commands"
#

BTW: this is not displayed when using "diff -wu".
Tough P5 it should still be corrected some time.

[Comment 1 SpanKY 2006-01-19 18:17:37 UTC]

you didnt post `emerge info`

[Comment 2 Roy Marples (RETIRED) 2006-01-20 02:47:30 UTC]

Thanks for the patch!

Fixed in baselayout-1.12.0_pre15

[Comment 3 Mark Dominik Bürkle 2006-01-23 08:29:39 UTC]

verified.

[Comment 4 Mark Dominik Bürkle 2006-10-11 08:46:14 UTC]

closed fixed - may be purged from DB.