Message-ID: <4F6C0B5F.10502@alpinelinux.org>
Date: Thu, 22 Mar 2012 22:34:23 -0700
From: Nathan Angelacos <nangel@alpinelinux.org>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.24) Gecko/20111114 Icedove/3.1.16
Precedence: list
MIME-Version: 1.0
To: alpine-devel@lists.alpinelinux.org
Subject: Re: [alpine-devel] snapshot for 120316
References: <20120316182256.7217b654@alpinelinux.org>	<4F69F6AA.7010006@alpinelinux.org> <20120321223736.30ff87e5@alpinelinux.org>
In-Reply-To: <20120321223736.30ff87e5@alpinelinux.org>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

>
>> Not sure that should be allowed - if not, perhaps change line 19 to
>> something like:
>>
>> *) echo "Interface name is not valid for vlan" ; exit 1;;
>
> actually, it is probably allowed to create vlan interfaces with any
> name using iproute2. I removed the code that does that uses it though
> since the busybox iproute does not support it. We could probably make
> it work by detecting if its bb iproute or not. (i wanted ship something
> working rather than aim for perfect and not ship at all)
>
> Care to file a bug so i don't forget it?

I think its good to only allow names like vlan[0-9]+

Alpine Linux is BusyBox based.   This script should be run when Alpine 
is first installed - iproute2 shouldn't be loaded.

If someone wants to load iproute2, they should be smart enough to 
configure the interfaces themselves.



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---