Re: ifconfig: remove obsolete vlan code

2019-04-15 Thread Stuart Henderson
On 2019/04/15 08:16, David Gwynne wrote: > > # ifconfig vlan0 | grep encap > > encap: vnetid none parent none txprio packet > > > > > > Shouldn't there be completely interoperability? If so, that seems like > > one more reason for removing the old interface, with or without

Re: ifconfig: remove obsolete vlan code

2019-04-14 Thread David Gwynne
> On 15 Apr 2019, at 05:56, Klemens Nanni wrote: > > On Sun, Apr 14, 2019 at 07:46:59PM +0200, Sebastian Benoit wrote: >> I dont mind keeping vlan/vlandev either, but then they should be aliases, >> not with their own function. > Fine with me as well. > > Diff below removes the old functions

Re: ifconfig: remove obsolete vlan code

2019-04-14 Thread Klemens Nanni
On Sun, Apr 14, 2019 at 07:46:59PM +0200, Sebastian Benoit wrote: > I dont mind keeping vlan/vlandev either, but then they should be aliases, > not with their own function. Fine with me as well. Diff below removes the old functions while keeping `[-]vlan' and `[-]vlandev' as aliases for

Re: ifconfig: remove obsolete vlan code

2019-04-14 Thread Sebastian Benoit
Stuart Henderson(s...@spacehopper.org) on 2019.04.14 11:35:57 +0100: > I still don't understand why we wouldn't just keep these as aliases to > parent/vnetid.. When parent/vnetid was introduced, the idea was to remove the old ones. Usually we clean things up eventually, otherwise in a year or

Re: ifconfig: remove obsolete vlan code

2019-04-14 Thread Stuart Henderson
I still don't understand why we wouldn't just keep these as aliases to parent/vnetid.. On 13 April 2019 19:41:33 Klemens Nanni wrote: `vlan' and `[-]vlandev' were deprecated starting with 6.3, 6.5 printed a warning when using it and current.html had a note as well, now is the time to remove

ifconfig: remove obsolete vlan code

2019-04-13 Thread Klemens Nanni
`vlan' and `[-]vlandev' were deprecated starting with 6.3, 6.5 printed a warning when using it and current.html had a note as well, now is the time to remove them. OK? Index: ifconfig.c === RCS file: