On 6/11/10 3:25 PM, "Steve Lawrence" <slawrence@xxxxxxxxxx> wrote: > On Mon, 2010-05-03 at 13:45 -0400, Daniel J Walsh wrote: >> Quality Engineering is going through all commands on the system looking >> for mismatches between man page/usage and actual code. >> >> It found that checkmodule had a -d option that is unused and undocumented -h > > Reviewed-by: Steve Lawrence <slawrence@xxxxxxxxxx> > > I'd just add the long --help option to the man page for completeness: > > diff --git a/checkpolicy/checkmodule.8 b/checkpolicy/checkmodule.8 > index a25f25c..54680e3 100644 > --- a/checkpolicy/checkmodule.8 > +++ b/checkpolicy/checkmodule.8 > @@ -25,7 +25,7 @@ the module package into the module store and load the > resulting policy. > Read an existing binary policy module file rather than a source policy > module file. This option is a development/debugging aid. > .TP > -.B \-h > +.B \-h,\-\-help > Print usage. > .TP > .B \-m > > > -- Acked-by: Chad Sellers <csellers@xxxxxxxxxx> Merged as of checkpolicy 2.0.22 -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
