[libvirt] [PATCH 1/1] Clarify what documentation is being referenced
Dave Allan
dallan at redhat.com
Tue Feb 28 15:37:37 UTC 2012
On Mon, Feb 27, 2012 at 04:02:44PM -0700, Eric Blake wrote:
> On 02/27/2012 03:51 PM, Dave Allan wrote:
> > virsh.pod had several instances in which it referred to "the
> > documentation" which was a little puzzling to me since it is
> > documentation. Reading the document from end to end makes it clear
> > that it means a specific URI which was noted previously in the text,
> > but I had never noticed those URI in several years of referring to the
> > man page. This patch adds those URIs to several additional places in
> > the text.
> > ---
> > tools/virsh.pod | 38 ++++++++++++++++++++++----------------
> > 1 files changed, 22 insertions(+), 16 deletions(-)
>
> Much nicer. ACK with one nit.
>
> > =item B<attach-device> I<domain-id> I<FILE>
> >
> > -Attach a device to the domain, using a device definition in an XML file.
> > -See the documentation to learn about libvirt XML format for a device.
> > -For cdrom and floppy devices, this command only replaces the media within
> > -the single existing device; consider using B<update-device> for this usage.
> > -For passthrough host devices, see also B<nodedev-dettach>, needed if
> > -the device does not use managed mode.
> > +Attach a device to the domain, using a device definition in an XML
> > +file. See the documentation at
> > +L<http://libvirt.org/formatdomain.html> to learn about libvirt XML
>
> Here, I'd link to formatdomain.html#elementsDevices, as well as
> specifically mention an example or two of the valid top-level elements
> we are looking for. Maybe:
>
> ...using a device definition (such as <disk> or <interface>) as the
> top-level element in an XML file. See the documentation...
Good point; I'll send a v2 shortly.
Dave
> --
> Eric Blake eblake at redhat.com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
More information about the libvir-list
mailing list