Re: [PATCH rt-tests 2/9] cyclictest: fixup documentation for --priority option

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

 



On Tue, Sep 01, 2015 at 11:03:08PM +0200, John Kacur wrote:
> On Mon, 31 Aug 2015, Josh Cartwright wrote:
> 
> > Commit 15c6d81986b6 ("[cyclictest] added priority spreading option
> > --priospread") introduced an additional option with the prefix 'prio';
> > this means that it is no longer possible for a user to specify '--prio'
> > instead of the full '--priority'.  Doing so results in the following
> > error:
> > 
> >   $ cyclictest --prio=4
> >   cyclictest: option '--prio=4' is ambiguous; possibilities: '--priority' '--priospread'
> > 
> > Fix this up by updating the 'help' output.
> > 
> > Signed-off-by: Josh Cartwright <joshc@xxxxxx>
> > ---
> >  src/cyclictest/cyclictest.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
> > index 9aebe32..4ced67f 100644
> > --- a/src/cyclictest/cyclictest.c
> > +++ b/src/cyclictest/cyclictest.c
> > @@ -1062,7 +1062,7 @@ static void display_help(int error)
> >  	       "-N       --nsecs           print results in ns instead of us (default us)\n"
> >  	       "-o RED   --oscope=RED      oscilloscope mode, reduce verbose output by RED\n"
> >  	       "-O TOPT  --traceopt=TOPT   trace option\n"
> > -	       "-p PRIO  --prio=PRIO       priority of highest prio thread\n"
> > +	       "-p PRIO  --priority=PRIO   priority of highest prio thread\n"
> >  	       "-P       --preemptoff      Preempt off tracing (used with -b)\n"
> >  	       "-q       --quiet           print only a summary on exit\n"
> >  	       "	 --priospread       spread priority levels starting at specified value\n"
> > -- 
> > 2.5.0
> 
> This was already fixed in v0.93
> 417ddb5db0ab cyclictest: Fix long priority help text option

Ahh, so it was.  Sorry about that, I had my git remote pointed at
clrkwllms/rt-tests.git instead of rt-tests/rt-tests.git which hadn't
been updated.

The rest should still be relevant.

Thanks,

  Josh

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux