Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 119629

Summary: rcscripts/net.modules.d/iwconfig wont find "${conf} " if not eval'ed
Product: Gentoo Linux Reporter: Mark Dominik Bürkle <dominik.buerkle>
Component: [OLD] baselayoutAssignee: Roy Marples (RETIRED) <uberlord>
Status: VERIFIED FIXED    
Severity: minor    
Priority: Lowest    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

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 gentoo-dev 2006-01-19 18:17:37 UTC
you didnt post `emerge info`
Comment 2 Roy Marples (RETIRED) gentoo-dev 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.