[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] doc: Add info (where necessary) that paths should be specified as absolute



On 04/07/2015 06:05 AM, Erik Skultety wrote:
> We documented this almost everywhere, but missed it on several places.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1208763
> ---
> 
> Hopefully I didn't miss any place where it should be fixed as well.
> I also thought about adding a check for this issue in the code, but we probably
> do not want to do that during parsing, as the existing domains might as well
> disappear. The other thing is, that if the path is invalid, "No such file or
> directory" is returned which in my opinion describes the problem quite well.
> 
>  docs/formatdomain.html.in  | 35 +++++++++++++++++++----------------
>  docs/formatstorage.html.in |  7 ++++---
>  2 files changed, 23 insertions(+), 19 deletions(-)

ACK. Even if you missed a spot, this is still a strict improvement.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]