Re: [PATCH] semodule: Fix the --list-modules=full example in the manpage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/18/2016 03:57 AM, Petr Lautrbach wrote:
> On 05/17/2016 05:27 PM, Laurent Bigonville wrote:
>> From: Laurent Bigonville <bigon@xxxxxxxx>
>> 
>> --- policycoreutils/semodule/semodule.8 | 2 +- 1 file changed, 1
>> insertion(+), 1 deletion(-)
>> 
>> diff --git a/policycoreutils/semodule/semodule.8
>> b/policycoreutils/semodule/semodule.8 index 68e95ab..7b07c45
>> 100644 --- a/policycoreutils/semodule/semodule.8 +++
>> b/policycoreutils/semodule/semodule.8 @@ -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-modules=full
> 
> 
> It works without " " before full:
> 
> $ semodule -lfull
> 
> What about:
> 
> policycoreutils/semodule/semodule.8:
> 
> .TP -.B  \-l,\-\-list-modules=[KIND] +.B
> \-l[KIND],\-\-list-modules[=KIND] display list of installed modules
> (other than base)
> 
> # List all modules. -$ semodule \-l full +$ semodule \-lfull # Set
> an alternate path for the policy root
> 
> 
> policycoreutils/semodule/semodule.c:
> 
> 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");

That works for me.  Someone want to spin an actual patch?  Although
the -l argument handling is somewhat odd and non-standard; if it
wasn't already in place, I think I'd have advocated for a separate
option flag if you want the full/long listing.
_______________________________________________
Selinux mailing list
Selinux@xxxxxxxxxxxxx
To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx.
To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.



[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux