[Libguestfs] [PATCH] pod: Use F<> for filenames instead of C<>.
Richard W.M. Jones
rjones at redhat.com
Mon Jun 15 14:43:10 UTC 2015
On Mon, Jun 15, 2015 at 04:06:09PM +0200, Pino Toscano wrote:
> On Sunday 14 June 2015 14:37:26 Richard W.M. Jones wrote:
> > --- a/generator/actions.ml
> > +++ b/generator/actions.ml
> > @@ -615,7 +615,7 @@ against.
> > Note that because of dynamic linking this is not necessarily
> > the version of libguestfs that you compiled against. You can
> > compile the program, and then at runtime dynamically link
> > -against a completely different C<libguestfs.so> library.
> > +against a completely different F<libguestfs.so> library.
> >
> > This call was added in version C<1.0.58>. In previous
> > versions of libguestfs there was no way to get the version
> > @@ -830,7 +830,7 @@ to specify the QEMU interface emulation to use at run time." };
> > ];
> > shortdesc = "detect the architecture of a binary file";
> > longdesc = "\
> > -This detects the architecture of the binary C<filename>,
> > +This detects the architecture of the binary F<filename>,
> > and returns it if known.
>
> (and other similar cases here in actions.ml)
>
> Considering that "filename" is the name of parameter, shouldn't it be
> better left with C<>?
It's a bit of a tricky corner case, but at least I changed all
filename parameters to F<...> (fairly) consistently :-)
> > @@ -81,9 +81,9 @@ C<VIRT_P2V_DATA_DIR> environment variable.
> >
> > =item C<$datadir/issue>
> >
> > -=item C<$datadir/launch-virt-p2v.in>
> > +=item F<$datadir/launch-virt-p2v.in>
> >
> > -=item C<$datadir/p2v.service>
> > +=item F<$datadir/p2v.service>
>
> I guess C<$datadir/issue> too?
Yup - fixed, thanks. There was another one in virt-p2v-make-kickstart
which I also fixed.
Rich.
> The rest seems fine to me.
>
> Thanks,
> --
> Pino Toscano
>
> _______________________________________________
> Libguestfs mailing list
> Libguestfs at redhat.com
> https://www.redhat.com/mailman/listinfo/libguestfs
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-builder quickly builds VMs from scratch
http://libguestfs.org/virt-builder.1.html
More information about the Libguestfs
mailing list