patch1/4: lvm manpages

patch1/4: lvm manpages

[Christoph Anton Mitterer]

Hi.

Attached would be several patches, that (IMHO) improve the lvm(8) manpage:



1) lvm.8.empty_LV.patch
It seems that the current "Where a list of VGs is required but is left empty, a
list  of  all VGs will be substituted." is also true for LVs (checked it with
lvdisplay, which seems to lists all LVs).
If this is true, add the patch, to reflect this in the documentation


Cheers,
Chris.




--- lvm.8	2010-06-23 16:25:44.099108000 +0200
+++ lvm.8	2010-06-23 16:28:42.675357517 +0200
@@ -24,8 +24,8 @@
 .LP
 Where commands take VG or LV names as arguments, the full path name is
 optional.  An LV called "lvol0" in a VG called "vg0" can be specified
-as "vg0/lvol0".  Where a list of VGs is required but is left empty, 
-a list of all VGs will be substituted.  Where a list of LVs is required
+as "vg0/lvol0".  Where a list of VGs or LVs is required but left empty, 
+a list of all VGs or LVs will be substituted.  Where a list of LVs is required
 but a VG is given, a list of all the LVs in that VG will be substituted.
 So "lvdisplay vg0" will display all the LVs in "vg0".
 Tags can also be used - see \fBaddtag\fP below.

patch1/4: lvm manpages

[Zdenek Kabelac]

Dne 20.8.2010 01:58, Christoph Anton Mitterer napsal(a):
> Hi.
> 
> Attached would be several patches, that (IMHO) improve the lvm(8) manpage:
> 
> 
> 
> 1) lvm.8.empty_LV.patch
> It seems that the current "Where a list of VGs is required but is left empty, a
> list  of  all VGs will be substituted." is also true for LVs (checked it with
> lvdisplay, which seems to lists all LVs).
> If this is true, add the patch, to reflect this in the documentation
> 

lvdisplay - with empty list will substitutes with all VGs - thus effectively
display all LVs from all VGs - but it doesn't mean there is some substitution
for LVs. It's the effect of command that it displays all LVs from given VG.
Lots of commands scans for all LVs in given VGs  (i.e. vgchange).

So this patch is rejected.

Zdenek

patch1/4: lvm manpages

[Christoph Anton Mitterer]

On Fri, 20 Aug 2010 15:09:05 +0200, Zdenek Kabelac <zkabelac@redhat.com>
wrote:
> lvdisplay - with empty list will substitutes with all VGs - thus
> effectively
> display all LVs from all VGs - but it doesn't mean there is some
> substitution
> for LVs. It's the effect of command that it displays all LVs from given
VG.
> Lots of commands scans for all LVs in given VGs  (i.e. vgchange).

Well... I still feel that there's something missing somewhere,...
nevertheless,... that's anyway just perfectionism =)

Cheers,
Chris.