[PATCH] Remove 'samba-tool user add'

Björn Baumbach bb at sernet.de
Thu Apr 30 10:07:48 UTC 2020


>> 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

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: 0551-370000-0, mailto:kontakt at sernet.de
Gesch.F.: Dr. Johannes Loxen und Reinhild Jung
AG Göttingen: HR-B 2816 - http://www.sernet.de

Samba eXPerience 2020 - online edition!
sponsored by Google, Microsoft & SerNet
May 26th-28th 2020, https://sambaXP.org



More information about the samba-technical mailing list