[lvm-devel] [PATCH] Document --all option in man pages, cleanup lvdisplay/pvdisplay man page.
Alasdair G Kergon
agk at redhat.com
Wed Oct 21 12:51:47 UTC 2009
On Tue, Oct 20, 2009 at 11:58:13PM -0400, Dave Wysochanski wrote:
> lvdisplay \- display attributes of a logical volume
> +.B lvdisplay \-\-columns | \-C
Still doesn't match the --help text, which shows which arguments
go with which form of the command.
> \fBlvs\fP (8) is an alternative that provides the same information
> in the style of \fBps\fP (1). \fBlvs\fP is recommended over
> -\fBlvdisplay\fP.
> +\fBlvdisplay\fP. \fBlvdisplay --columns\fP is the equivalent
> +of calling \fBlvs\fP.
Mention that under -C in the OPTIONS section.
"Calling X is the equivalent of calling Y" ('Running' or 'Executing'?)
or better, just "X is equivalent to Y".
> .B lvs
> +[\-a|\-\-all]
> +Process all logical volumes, even hidden/internal ones.
Replace 'process' with a more descriptive verb based on what
'lvs' does.
Replace 'hidden/internal' with a better description of what
is meant. I can spot no existing references to 'internal'
in this context in the man pages, and only one to 'implemented by
creating a hidden virtual device'.
Alasdair
More information about the lvm-devel
mailing list