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? The point of hiding the option is to make it less discoverable. _______________________________________________ sssd-devel mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
