[PATCH] Remove 'samba-tool user add'
Karolin Seeger
kseeger at samba.org
Tue May 5 09:57:25 UTC 2020
Hi Björn,
Am 30.04.20 um 12:07 schrieb Björn Baumbach via samba-technical:
>>> 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 :-)
first of all, thanks for taking care! :-)
I would prefer "2" to make sure that the user interface does not change
(existing scripts...).
Cheers,
Karo
--
Karolin Seeger https://samba.org/~kseeger/
Release Manager Samba Team https://samba.org
Team Lead Samba SerNet https://sernet.de
More information about the samba-technical
mailing list