[PATCH] group.py: Fix wrong example option, remove wrong comment line
Michael Adam
obnox at samba.org
Wed Jun 3 03:59:06 MDT 2015
Hi Marc,
the change is good, but I'd encourage you to split
it into two commits: They are only related by changing
the same file.
If a commit message reads "do foo AND bar"
is an indication that you want it to be two
commits "do foo" and "do bar", is my mantra here.
I'll re-review/push when you resubmit.
Cheers - Michael
On 2015-06-03 at 07:41 +0200, Marc Muehlfeld wrote:
> Hello,
>
> attached a patch, with two tiny fixes for python/samba/netcmd/group.py.
>
> Please review and push.
>
> Thanks to Rowland, for finding that. I created a bug report, to track
> this: https://bugzilla.samba.org/show_bug.cgi?id=11072
>
> Regards,
> Marc
>
>
>
>
> Am 24.01.2015 um 13:51 schrieb Rowland Penny:
> > The other thing that is puzzling me is '--gid=12345', yet in the
> > script there is this:
> >
> > Option("--gid-number", help="Group's Unix/RFC2307 GID number",
> > type=int),
> >
> > ....
> >
> > Just one last comment, the first line of group.py is this:
> >
> > # Adds a new user to a Samba4 server
> >
> > ????
> >
> > Rowland
> >From b15278afd88de29d88d143433e732e797e11357a Mon Sep 17 00:00:00 2001
> From: Marc Muehlfeld <mmuehlfeld at samba.org>
> Date: Sat, 24 Jan 2015 15:59:40 +0100
> Subject: [PATCH] group.py: Fix wrong example option, remove wrong comment line
>
> Replaced "--gid" with the correct "--gid-number" in Example 3.
> Additionally removed the first comment line in group.py, which
> was wrong in that file.
>
> Signed-off-by: Marc Muehlfeld <mmuehlfeld at samba.org>
> ---
> python/samba/netcmd/group.py | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/python/samba/netcmd/group.py b/python/samba/netcmd/group.py
> index 4b5fd27..722bcc4 100644
> --- a/python/samba/netcmd/group.py
> +++ b/python/samba/netcmd/group.py
> @@ -1,4 +1,3 @@
> -# Adds a new user to a Samba4 server
> # Copyright Jelmer Vernooij 2008
> #
> # Based on the original in EJS:
> @@ -72,7 +71,7 @@ sudo samba-tool group add Group2 --group-type=Distribution
> Example2 adds a new distribution group to the local server. The command is run under root using the sudo command.
>
> Example3:
> -samba-tool group add Group3 --nis-domain=samdom --gid=12345
> +samba-tool group add Group3 --nis-domain=samdom --gid-number=12345
>
> Example3 adds a new RFC2307 enabled group for NIS domain samdom and GID 12345 (both options are required to enable this feature).
> """
> --
> 2.1.0
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20150603/9922b4b6/attachment.pgp>
More information about the samba-technical
mailing list