netdev - Re: [PATCH iproute2 0/5] Eliminate the term slave in iproute2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200624095142.zete44mbrtieepju@lion.mk-sys.cz>
Date:   Wed, 24 Jun 2020 11:51:42 +0200
From:   Michal Kubecek <mkubecek@...e.cz>
To:     netdev@...r.kernel.org
Cc:     Stephen Hemminger <stephen@...workplumber.org>
Subject: Re: [PATCH iproute2 0/5] Eliminate the term slave in iproute2

On Tue, Jun 23, 2020 at 04:53:02PM -0700, Stephen Hemminger wrote:
> These patches remove the term slave from the iproute2 visible
> command line, documentation, and variable naming.
> 
> This needs doing despite the fact it will cause cosmetic
> changes to visible outputs.

AFAICS, your patches don't only change output of the command, they also
change command line parsing so that they break backward compatibility
and break existing scripts and tools. This is a very bad idea.

If you really want to go through this exercise - and I don't agree with
the "needs doing" claim - you definitely should preserve existing
keywords as aliases.

Michal