On 27 August 2010 00:09, Doug Barton <dougb@freebsd.org> wrote:
quoted text
> On 08/26/2010 12:53 PM, pluknet wrote:
>>
>> [cc'ing current@ as rc@ looks too quite]
>>
>> Hi.
>>
>> Since ifconfig has grown to label interfaces with
>> ifconfig $ifname description "foobar", what about
>> to give it more life and store i/face descriptions
>> semi-permanently, so they will survive between reboots?
>>
>> This patch adds a functionality to rc.d to label
>> interfaces at boot time.
>>
>> Comments are welcome.
>
> This seems like a good addition, thanks. Please also write a patch for
> rc.conf.5 to describe this new functionality and I'll be happy to commit it.

Xin Li helped me with updating rc.conf.5 (thanks!).
It's included in attached patch.

quoted text
>  One note below.
>
>
>> --- etc/network.subr    (revision 211280)
>> +++ etc/network.subr    (working copy)
>> @@ -1187,6 +1187,24 @@
>>         return 0
>>  }
>>
>> +# ifnet_descr
>> +#      Add description to all requested interfaces.
>> +#
>> +ifnet_descr()
>> +{
>> +       local _if _ifdescr
>> +
>> +       # ifconfig_IF_descr
>> +       for _if in `ifconfig -l`; do
>> +               _ifdescr="`get_if_var $_if ifconfig_IF_descr`"
>> +               if [ ! -z "$_ifdescr" ]; then
>
> This is probably better as [ -n "$_ifdescr" ]
>

This was blindly copy&pasted after ifnet_rename().
So, it makes sense probably to change test expression there as well.
[see ifnet_rename() proposed change below inline]
This change to ifnet_rename() is not included in attached patch
to not complicate things unnecessarily for now.

Index: etc/network.subr
===================================================================
--- etc/network.subr    (revision 211280)
+++ etc/network.subr    (working copy)
@@ -1179,7 +1179,7 @@
        # ifconfig_IF_name
        for _if in `ifconfig -l`; do
                _ifname=`get_if_var $_if ifconfig_IF_name`
-               if [ ! -z "$_ifname" ]; then
+               if [ -n "$_ifname" ]; then
                        ifconfig $_if name $_ifname
                fi
        done

-- 
wbr,
pluknet