[PATCH] Remove 'samba-tool user add'

[Rowland penny]

On 30/04/2020 11:07, Björn Baumbach wrote:
>>> Andrew Bartlett via samba-technical <samba-technical at lists.samba.org>
>>> wrote:
>>>> Can someone remind me why it was deprecated?
> On 3/29/17 3:04 PM, Volker Lendecke via samba-technical wrote:
>> Also, shouldn't we align this with the group commands? There we have a
>> "samba-tool group add", but no "samba-tool group create".
> Hi!
>
> I would like to continue this proposal and like to read your ideas to
> resolve this.
>
> I've recently opened a merge request with a set of patches which rename
> the "samba-tool group add" command to "samba-tool group create" and
> deprecated the add:
> https://gitlab.com/samba-team/samba/-/merge_requests/1304
>
> I planned to fix all other "add" subcommands, but I've noticed that most
> of the subcommands like computer, contact, ou, ... use "create" already.
>
> In addition to the "group add" command there are two further "add"s
> left, but they make really sense, in my opinion. It's "spn add" to add a
> SPN to an account (while the command description says "add - *Create* a
> new spn") and "dns add" to add a DNS record.
>
> I think the "add" command is generally more comfortable and seems to be
> more natural, even for adding new users or groups to the AD.
>
> It seems that several people wonder why the "user add" command has been
> renamed to "user create". Unfortunately the commit message does not
> provide this information:
>
> commit b26a4f6232219746fd8ed1b107476f3c23c6908e
> Author: Theresa Halloran <thallora at linux.vnet.ibm.com>
> Date:   Sun Oct 23 15:23:28 2011 -0700
>
>      s4: samba-tool subcommand rename - change samba-tool user add to
> samba-tool user create
>
>      Signed-off-by: Theresa Halloran <thallora at linux.vnet.ibm.com>
>      Signed-off-by: Jelmer Vernooij <jelmer at samba.org>
>
>
> What do you think about:
> 1.) Just replace the "group add" with "group create" (and keep "dns add"
>      and "spn add"), like in my merge request.
> 2.) Have both, "add" and "create", for all subcommands.
> 3.) Roll-back: Remove the "user add" deprecation warning, add the "add"
>      command to all other categories and deprecate all "create" commands.
>
> Please let me know :-)
>
> Best regards,
> Björn
>
The problem with '1' is that there  will undoubtedly be scripts out 
there that use 'group add', so you will have to deprecate 'group add' 
then remove it at a later date and just look at how long it has taken to 
remove 'user add'.

'2' will just lead to confusion.

'3' will mean deprecating 'user create' and this will just lead to 
confusion.

I would just remove 'user add'.

Rowland