[libvirt] [PATCH] man: virsh: Add man page for "virsh metadata"

Peter Krempa pkrempa at redhat.com
Mon Aug 18 08:36:14 UTC 2014


On 08/15/14 18:22, Eric Blake wrote:
> On 08/15/2014 09:12 AM, Peter Krempa wrote:
>> Patch adding the command forgot to add the man page entry.
>>
>> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1130379
>> ---
>>  tools/virsh.pod | 30 ++++++++++++++++++++++++++++++
>>  1 file changed, 30 insertions(+)
>>
>> diff --git a/tools/virsh.pod b/tools/virsh.pod
>> index f07deec..d072a7f 100644
>> --- a/tools/virsh.pod
>> +++ b/tools/virsh.pod
>> @@ -1254,6 +1254,36 @@ I<--total> for only the total stats, I<start> for only the per-cpu
>>  stats of the CPUs from I<start>, I<count> for only I<count> CPUs'
>>  stats.
>>
>> +=item B<metadata> I<domain> [[I<--live>] [I<--config>] | [I<--current>]]
>> +[I<--edit>] [I<uri>] [I<key>] [I<set>] [I<--remove>]
> 
> Looks good, but one suggestion (possibly as a followup patch):
> 
> I think =item B<desc> and =item B<metadata> should have cross-references
> to one another, since we now have two separate virsh commands that both
> map to the same underlying libvirt API.
> 
> ACK
> 

Pushed; Thanks.

I'll find out how to do references in .pod and propose a patch for that.

Peter

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140818/c8c7c37b/attachment-0001.sig>


More information about the libvir-list mailing list