All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] blktrace: Fix documentation for version flag
@ 2008-10-15  9:35 Christof Schmitt
  2008-10-16  6:04 ` Jens Axboe
  2008-10-16  8:12 ` Christof Schmitt
  0 siblings, 2 replies; 3+ messages in thread
From: Christof Schmitt @ 2008-10-15  9:35 UTC (permalink / raw)
  To: linux-btrace

blktrace uses the -V flag (uppercase) for printing the version
information. Fix the man page and the usage information that wrongly
state the -v flag (lowercase).

Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
---
 blktrace.c     |    2 +-
 doc/blktrace.8 |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

--- a/blktrace.c	2008-10-15 11:12:53.000000000 +0200
+++ b/blktrace.c	2008-10-15 11:14:57.000000000 +0200
@@ -1756,7 +1756,7 @@ static int net_setup_client(void)
 
 static char usage_str[] = \
 	"-d <dev> [ -r debugfs path ] [ -o <output> ] [-k ] [ -w time ]\n" \
-	"[ -a action ] [ -A action mask ] [ -I  <devs file> ] [ -v ]\n\n" \
+	"[ -a action ] [ -A action mask ] [ -I  <devs file> ] [ -V ]\n\n" \
 	"\t-d Use specified device. May also be given last after options\n" \
 	"\t-r Path to mounted debugfs, defaults to /sys/kernel/debug\n" \
 	"\t-o File(s) to send output to\n" \
--- a/doc/blktrace.8	2008-10-15 11:12:53.000000000 +0200
+++ b/doc/blktrace.8	2008-10-15 11:15:22.000000000 +0200
@@ -6,7 +6,7 @@ blktrace \- generate traces of the i/o t
 
 
 .SH SYNOPSIS
-.B blktrace \-d \fIdev\fR [ \-r \fIdebugfs_path\fR ] [ \-o \fIoutput\fR ] [\-k ] [ \-w \fItime\fR ] [ \-a \fIaction\fR ] [ \-A \fIaction_mask\fR ] [ \-v ]
+.B blktrace \-d \fIdev\fR [ \-r \fIdebugfs_path\fR ] [ \-o \fIoutput\fR ] [\-k ] [ \-w \fItime\fR ] [ \-a \fIaction\fR ] [ \-A \fIaction_mask\fR ] [ \-V ]
 .br
 
 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] blktrace: Fix documentation for version flag
  2008-10-15  9:35 [PATCH] blktrace: Fix documentation for version flag Christof Schmitt
@ 2008-10-16  6:04 ` Jens Axboe
  2008-10-16  8:12 ` Christof Schmitt
  1 sibling, 0 replies; 3+ messages in thread
From: Jens Axboe @ 2008-10-16  6:04 UTC (permalink / raw)
  To: linux-btrace

On Wed, Oct 15 2008, Christof Schmitt wrote:
> blktrace uses the -V flag (uppercase) for printing the version
> information. Fix the man page and the usage information that wrongly
> state the -v flag (lowercase).
> 
> Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
> ---
>  blktrace.c     |    2 +-
>  doc/blktrace.8 |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> --- a/blktrace.c	2008-10-15 11:12:53.000000000 +0200
> +++ b/blktrace.c	2008-10-15 11:14:57.000000000 +0200
> @@ -1756,7 +1756,7 @@ static int net_setup_client(void)
>  
>  static char usage_str[] = \
>  	"-d <dev> [ -r debugfs path ] [ -o <output> ] [-k ] [ -w time ]\n" \
> -	"[ -a action ] [ -A action mask ] [ -I  <devs file> ] [ -v ]\n\n" \
> +	"[ -a action ] [ -A action mask ] [ -I  <devs file> ] [ -V ]\n\n" \
>  	"\t-d Use specified device. May also be given last after options\n" \
>  	"\t-r Path to mounted debugfs, defaults to /sys/kernel/debug\n" \
>  	"\t-o File(s) to send output to\n" \
> --- a/doc/blktrace.8	2008-10-15 11:12:53.000000000 +0200
> +++ b/doc/blktrace.8	2008-10-15 11:15:22.000000000 +0200
> @@ -6,7 +6,7 @@ blktrace \- generate traces of the i/o t
>  
>  
>  .SH SYNOPSIS
> -.B blktrace \-d \fIdev\fR [ \-r \fIdebugfs_path\fR ] [ \-o \fIoutput\fR ] [\-k ] [ \-w \fItime\fR ] [ \-a \fIaction\fR ] [ \-A \fIaction_mask\fR ] [ \-v ]
> +.B blktrace \-d \fIdev\fR [ \-r \fIdebugfs_path\fR ] [ \-o \fIoutput\fR ] [\-k ] [ \-w \fItime\fR ] [ \-a \fIaction\fR ] [ \-A \fIaction_mask\fR ] [ \-V ]
>  .br

I'll just change blktrace to take -v as well, that is what most other
programs accept so no point in deviating there.

-- 
Jens Axboe


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] blktrace: Fix documentation for version flag
  2008-10-15  9:35 [PATCH] blktrace: Fix documentation for version flag Christof Schmitt
  2008-10-16  6:04 ` Jens Axboe
@ 2008-10-16  8:12 ` Christof Schmitt
  1 sibling, 0 replies; 3+ messages in thread
From: Christof Schmitt @ 2008-10-16  8:12 UTC (permalink / raw)
  To: linux-btrace

On Thu, Oct 16, 2008 at 08:04:06AM +0200, Jens Axboe wrote:
> I'll just change blktrace to take -v as well, that is what most other
> programs accept so no point in deviating there.

That is also fine for me. I assumed changing the documentation would
be simpler.

Christof

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-10-16  8:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-15  9:35 [PATCH] blktrace: Fix documentation for version flag Christof Schmitt
2008-10-16  6:04 ` Jens Axboe
2008-10-16  8:12 ` Christof Schmitt

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.