diff mbox series

docs/devel/testing: Add -a option to usermod command on docker setup

Message ID 20190207184346.6840-1-muriloo@linux.ibm.com (mailing list archive)
State New, archived
Headers show
Series docs/devel/testing: Add -a option to usermod command on docker setup | expand

Commit Message

Murilo Opsfelder Araújo Feb. 7, 2019, 6:43 p.m. UTC
The option -G of usermod command will remove user from other groups
not listed, i.e.: $USER will belong only to group 'docker' after
following the documentation as is.

From usermod(8) manual page:

    If the user is currently a member of a group which is not listed,
    the user will be removed from the group. This behaviour can be
    changed via the -a option, which appends the user to the current
    supplementary group list.

This patch improves the situation by adding the -a option to the
usermod command, which will just append user to the supplementary
group list.

Cc: qemu-trivial@nongnu.org
Signed-off-by: Murilo Opsfelder Araujo <muriloo@linux.ibm.com>
---
 docs/devel/testing.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alex Bennée Feb. 8, 2019, 5:32 p.m. UTC | #1
Murilo Opsfelder Araujo <muriloo@linux.ibm.com> writes:

> The option -G of usermod command will remove user from other groups
> not listed, i.e.: $USER will belong only to group 'docker' after
> following the documentation as is.
>
> From usermod(8) manual page:
>
>     If the user is currently a member of a group which is not listed,
>     the user will be removed from the group. This behaviour can be
>     changed via the -a option, which appends the user to the current
>     supplementary group list.
>
> This patch improves the situation by adding the -a option to the
> usermod command, which will just append user to the supplementary
> group list.
>
> Cc: qemu-trivial@nongnu.org
> Signed-off-by: Murilo Opsfelder Araujo <muriloo@linux.ibm.com>

Queued to testing/next, thanks.

> ---
>  docs/devel/testing.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index 18e2c0868a..135743a2bf 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -301,7 +301,7 @@ An alternative method to set up permissions is by adding the current user to
>  .. code::
>
>    $ sudo groupadd docker
> -  $ sudo usermod $USER -G docker
> +  $ sudo usermod $USER -a -G docker
>    $ sudo chown :docker /var/run/docker.sock
>
>  Note that any one of above configurations makes it possible for the user to


--
Alex Bennée
diff mbox series

Patch

diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index 18e2c0868a..135743a2bf 100644
--- a/docs/devel/testing.rst
+++ b/docs/devel/testing.rst
@@ -301,7 +301,7 @@  An alternative method to set up permissions is by adding the current user to
 .. code::
 
   $ sudo groupadd docker
-  $ sudo usermod $USER -G docker
+  $ sudo usermod $USER -a -G docker
   $ sudo chown :docker /var/run/docker.sock
 
 Note that any one of above configurations makes it possible for the user to