[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 05:48 PM, Eric Blake wrote:
> 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.
> 
Thank you, I pushed the patch a couple of hours ago.
Erik


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