diff mbox

policycoreutils/semodule: Fix the documentation of -l,--list

Message ID 1464076949-27068-1-git-send-email-plautrba@redhat.com (mailing list archive)
State Not Applicable
Headers show

Commit Message

Petr Lautrbach May 24, 2016, 8:02 a.m. UTC
It wasn't clear how to use -l option with full/standard KIND and the
example in semodule.8 man page was wrong.

Based on a patch by Laurent Bigonville <bigon@bigon.be>
Signed-off-by: Petr Lautrbach <plautrba@redhat.com>
---
 policycoreutils/semodule/semodule.8 | 4 ++--
 policycoreutils/semodule/semodule.c | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

Comments

James Carter June 6, 2016, 5:12 p.m. UTC | #1
On 05/24/2016 04:02 AM, Petr Lautrbach wrote:
> It wasn't clear how to use -l option with full/standard KIND and the
> example in semodule.8 man page was wrong.
>
> Based on a patch by Laurent Bigonville <bigon@bigon.be>
> Signed-off-by: Petr Lautrbach <plautrba@redhat.com>

Applied.

Thanks,

> ---
>  policycoreutils/semodule/semodule.8 | 4 ++--
>  policycoreutils/semodule/semodule.c | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/policycoreutils/semodule/semodule.8 b/policycoreutils/semodule/semodule.8
> index 68e95ab..0c5fdf7 100644
> --- a/policycoreutils/semodule/semodule.8
> +++ b/policycoreutils/semodule/semodule.8
> @@ -38,7 +38,7 @@ deprecated, alias for --install
>  .B  \-r,\-\-remove=MODULE_NAME
>  remove existing module
>  .TP
> -.B  \-l,\-\-list-modules=[KIND]
> +.B  \-l[KIND],\-\-list-modules[=KIND]
>  display list of installed modules (other than base)
>  .TP
>  .B  \-E,\-\-extract=MODULE_PKG
> @@ -116,7 +116,7 @@ $ semodule \-d alsa
>  # Install a module at a specific priority.
>  $ semodule \-X 100 \-i alsa.pp
>  # List all modules.
> -$ semodule \-l full
> +$ semodule \-\-list=full
>  # Set an alternate path for the policy root
>  $ semodule \-B \-p "/tmp"
>  # Set an alternate path for the policy store root
> diff --git a/policycoreutils/semodule/semodule.c b/policycoreutils/semodule/semodule.c
> index bcfaa2b..63900f4 100644
> --- a/policycoreutils/semodule/semodule.c
> +++ b/policycoreutils/semodule/semodule.c
> @@ -127,7 +127,7 @@ static void usage(char *progname)
>  	printf("  -B, --build		    build and reload policy\n");
>  	printf("  -i,--install=MODULE_PKG   install a new module\n");
>  	printf("  -r,--remove=MODULE_NAME   remove existing module\n");
> -	printf("  -l,--list-modules=[KIND]  display list of installed modules\n");
> +	printf("  -l[KIND],--list-modules[=KIND]  display list of installed modules\n");
>  	printf("     KIND:  standard  list highest priority, enabled modules\n");
>  	printf("            full      list all modules\n");
>  	printf("  -X,--priority=PRIORITY    set priority for following operations (1-999)\n");
>
diff mbox

Patch

diff --git a/policycoreutils/semodule/semodule.8 b/policycoreutils/semodule/semodule.8
index 68e95ab..0c5fdf7 100644
--- a/policycoreutils/semodule/semodule.8
+++ b/policycoreutils/semodule/semodule.8
@@ -38,7 +38,7 @@  deprecated, alias for --install
 .B  \-r,\-\-remove=MODULE_NAME
 remove existing module
 .TP
-.B  \-l,\-\-list-modules=[KIND]
+.B  \-l[KIND],\-\-list-modules[=KIND]
 display list of installed modules (other than base)
 .TP
 .B  \-E,\-\-extract=MODULE_PKG
@@ -116,7 +116,7 @@  $ semodule \-d alsa
 # Install a module at a specific priority.
 $ semodule \-X 100 \-i alsa.pp
 # List all modules.
-$ semodule \-l full
+$ semodule \-\-list=full
 # Set an alternate path for the policy root
 $ semodule \-B \-p "/tmp"
 # Set an alternate path for the policy store root
diff --git a/policycoreutils/semodule/semodule.c b/policycoreutils/semodule/semodule.c
index bcfaa2b..63900f4 100644
--- a/policycoreutils/semodule/semodule.c
+++ b/policycoreutils/semodule/semodule.c
@@ -127,7 +127,7 @@  static void usage(char *progname)
 	printf("  -B, --build		    build and reload policy\n");
 	printf("  -i,--install=MODULE_PKG   install a new module\n");
 	printf("  -r,--remove=MODULE_NAME   remove existing module\n");
-	printf("  -l,--list-modules=[KIND]  display list of installed modules\n");
+	printf("  -l[KIND],--list-modules[=KIND]  display list of installed modules\n");
 	printf("     KIND:  standard  list highest priority, enabled modules\n");
 	printf("            full      list all modules\n");
 	printf("  -X,--priority=PRIORITY    set priority for following operations (1-999)\n");