On (29/07/16 14:27), Jakub Hrozek wrote:
On Fri, Jul 29, 2016 at 02:09:02PM +0200, Lukas Slebodnik wrote:
> On (29/07/16 13:59), Jakub Hrozek wrote:
> >On Fri, Jul 29, 2016 at 01:49:41PM +0200, Lukas Slebodnik wrote:
> >> On (29/07/16 13:44), Jakub Hrozek wrote:
> >> >On Fri, Jul 29, 2016 at 01:07:56PM +0200, Lukas Slebodnik wrote:
> >> >> >Others who? :-)
> >> >> non developers (The person who requested this change; I assume
this
> >> >> change was not requested by developers)
> >> >
> >> >It was (and btw I agree with the change, consistent naming is important
> >> >as I wish I raised this concern when I reviewed the patches in the
first
> >> >place..)
> >> I was expecting an answer for keeping backward compatibility with
> >> unused feature.
> >
> >At this point it would be only compatibility for rawhide users and anyone
> >who compiled sssd from source or anyone who was alrady using the 1.14.0
> >tarball. Which is not many people, but still.
> So if you want to keep old versions then
> we document obsoleted version.
> At least with help "Deprecated alias for (new-name)"
>
> e.g.
> [root@host ~]# sssctl
> Usage:
> sssctl COMMAND COMMAND-ARGS
>
> Available commands:
>
> SSSD Status:
> * list-domains Deprecated alias for (domain-list)
> * domain-list List available domains
> * domain-status Print information about domain
>
> The idea of hidding options is really terrible.
> a) it's not documented anywhere that it's deprecated
> b) users might wonder why it works.
Fine by me, but additioanlly, what about printing the deprecation
warning when a user runs that command?
If we really want to insist on "backward
compatibility" with unused
feature then it will be good addition to the the updated help output.
The point of hiding the option is
to make it less discoverable.
I know what is point of hiding the option but it
isn't good from
user point of view if we want to keep backward compatibility.
Backward compatible changes are usually well documented and not hiden
Please update design page with renamed commands and also with
deprecated commands.
LS