Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-02-06 Thread Stefano Brivio
On Tue, 5 Feb 2019 14:50:33 -0800 Stephen Hemminger wrote: > Providing brctl or ifconfig scripts is possible, but it really should > be put in a sample or demo directory and not installed by default. > It would just cause too much pain to distributions. On one hand, I did it this way exactly to

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-02-05 Thread Stephen Hemminger
On Thu, 31 Jan 2019 08:28:29 -0800 Roopa Prabhu wrote: > On Thu, Jan 31, 2019 at 4:46 AM Stefano Brivio wrote: > > > > On Wed, 30 Jan 2019 14:30:59 -0800 > > Roopa Prabhu wrote: > > > > > On Sun, Jan 27, 2019 at 11:57 PM Stefano Brivio > > > wrote: > > > > > > > > They can't replace brctl

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-31 Thread Roopa Prabhu
On Thu, Jan 31, 2019 at 4:46 AM Stefano Brivio wrote: > > On Wed, 30 Jan 2019 14:30:59 -0800 > Roopa Prabhu wrote: > > > On Sun, Jan 27, 2019 at 11:57 PM Stefano Brivio wrote: > > > > > > They can't replace brctl not because they are badly designed or > > > unusable, but simply because they are

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-31 Thread Stefano Brivio
Hi, For your information: I recently submitted a patch that implements brctl as a wrapper using ip-link and 'bridge' from iproute2, that allows to drop bridge-utils while still providing brctl functionality: https://patchwork.ozlabs.org/patch/1027627/ This wasn't exactly met with enthusi

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-31 Thread Stefano Brivio
On Wed, 30 Jan 2019 14:30:59 -0800 Roopa Prabhu wrote: > On Sun, Jan 27, 2019 at 11:57 PM Stefano Brivio wrote: > > > > They can't replace brctl not because they are badly designed or > > unusable, but simply because they are different tools with different > > purposes (see also my comments to N

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-31 Thread Stefano Brivio
On Wed, 30 Jan 2019 22:12:45 -0700 David Ahern wrote: > This is a convenience wrapper around commands packaged in iproute2. If > iproute2 adds this wrapper, it will have to carry it and maintain it > forever. Distributions (Fedora, RHEL, Debian, etc) may see it > differently and decide to add thi

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-30 Thread David Ahern
On 1/30/19 3:55 AM, Stefano Brivio wrote: >> I get your intent, but this seems more appropriate for you / Red Hat to >> carry than something we want to distribute as part of iproute2. > > Sure, I could also do that, but: > > - me creating another project: similar maintenance burden for > distri

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-30 Thread Roopa Prabhu
On Sun, Jan 27, 2019 at 11:57 PM Stefano Brivio wrote: > > On Sun, 27 Jan 2019 21:08:13 -0800 > Roopa Prabhu wrote: > > > On Fri, Jan 25, 2019 at 2:05 AM Stefano Brivio wrote: > > > > > > Hi Roopa, > > > > > > On Wed, 23 Jan 2019 08:33:27 -0800 > > > Roopa Prabhu wrote: > > > > > > > On Wed, Ja

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-30 Thread Stefano Brivio
Hi David, On Tue, 29 Jan 2019 21:51:24 -0700 David Ahern wrote: > On 1/18/19 10:00 AM, Stefano Brivio wrote: > > This script wraps 'ip' and 'bridge' tools to provide a drop-in replacement > > of the standalone 'brctl' utility. > > > > It's bug-to-bug compatible with brctl as of bridge-utils ver

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-29 Thread David Ahern
On 1/18/19 10:00 AM, Stefano Brivio wrote: > This script wraps 'ip' and 'bridge' tools to provide a drop-in replacement > of the standalone 'brctl' utility. > > It's bug-to-bug compatible with brctl as of bridge-utils version 1.6, > has no dependencies other than a POSIX shell, and it's less than

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-27 Thread Stefano Brivio
On Sun, 27 Jan 2019 21:08:13 -0800 Roopa Prabhu wrote: > On Fri, Jan 25, 2019 at 2:05 AM Stefano Brivio wrote: > > > > Hi Roopa, > > > > On Wed, 23 Jan 2019 08:33:27 -0800 > > Roopa Prabhu wrote: > > > > > On Wed, Jan 23, 2019 at 7:09 AM Nikolay Aleksandrov > > > wrote: > > > > > > > Hi, >

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-27 Thread Roopa Prabhu
On Fri, Jan 25, 2019 at 2:05 AM Stefano Brivio wrote: > > Hi Roopa, > > On Wed, 23 Jan 2019 08:33:27 -0800 > Roopa Prabhu wrote: > > > On Wed, Jan 23, 2019 at 7:09 AM Nikolay Aleksandrov > > wrote: > > > > > Hi, > > > IMO the effort should be towards improving iproute2 to be > > > easier to use

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-25 Thread Stefano Brivio
Hi Roopa, On Wed, 23 Jan 2019 08:33:27 -0800 Roopa Prabhu wrote: > On Wed, Jan 23, 2019 at 7:09 AM Nikolay Aleksandrov > wrote: > > > Hi, > > IMO the effort should be towards improving iproute2 to be > > easier to use and more intuitive. We should be pushing people to > > use the new tools inst

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-25 Thread Stefano Brivio
Hi Nik, On Wed, 23 Jan 2019 17:09:42 +0200 Nikolay Aleksandrov wrote: > IMO the effort should be towards improving iproute2 to be > easier to use and more intuitive. We should be pushing people to use > the new tools instead of trying to find workarounds to keep the old > tools alive. Indeed, i

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-23 Thread Roopa Prabhu
On Wed, Jan 23, 2019 at 7:09 AM Nikolay Aleksandrov wrote: > > On 18/01/2019 19:00, Stefano Brivio wrote: > > This script wraps 'ip' and 'bridge' tools to provide a drop-in replacement > > of the standalone 'brctl' utility. > > > > It's bug-to-bug compatible with brctl as of bridge-utils version 1

Re: [PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-23 Thread Nikolay Aleksandrov
On 18/01/2019 19:00, Stefano Brivio wrote: > This script wraps 'ip' and 'bridge' tools to provide a drop-in replacement > of the standalone 'brctl' utility. > > It's bug-to-bug compatible with brctl as of bridge-utils version 1.6, > has no dependencies other than a POSIX shell, and it's less than

[PATCH iproute2-next] Introduce ip-brctl shell script

2019-01-18 Thread Stefano Brivio
This script wraps 'ip' and 'bridge' tools to provide a drop-in replacement of the standalone 'brctl' utility. It's bug-to-bug compatible with brctl as of bridge-utils version 1.6, has no dependencies other than a POSIX shell, and it's less than half the binary size of brctl on x86_64. As many use