[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v4 1/2] iotests: Use absolute paths for executab
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v4 1/2] iotests: Use absolute paths for executables |
Date: |
Sun, 2 Jul 2017 15:35:51 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
On 2017-06-30 21:41, Eric Blake wrote:
> On 06/29/2017 09:46 PM, Max Reitz wrote:
>
>>>> +++ b/tests/qemu-iotests/common.config
>>>> @@ -103,6 +103,12 @@ if [ -z "$QEMU_VXHS_PROG" ]; then
>>>> export QEMU_VXHS_PROG="`set_prog_path qnio_server`"
>>>> fi
>>>>
>>>> +export QEMU_PROG=$(realpath -- "$(type -p "$QEMU_PROG")")
>>>
>>> ...now that you updated per my review to favor 'type' over 'which'?
>>> Otherwise, the R-b stands.
>>
>> Thanks, will fix and apply then...
>>
>> ...and done, applied to my block branch:
>>
>> https://github.com/XanClic/qemu/commits/block
>
> Sorry for not noticing sooner, but you'll need to replace v4 (commit
> 0f0fec82 on your branch) with a fix, because now your branch does the
> following on all iotests for me:
>
> 068 3s ... - output mismatch (see 068.out.bad)
> --- /home/eblake/qemu/tests/qemu-iotests/068.out 2017-06-26
> 22:02:56.057734882 -0500
> +++ 068.out.bad 2017-06-30 14:35:28.720241398 -0500
> @@ -1,4 +1,5 @@
> QA output created by 068
> +realpath: '': No such file or directory
Oops.
> The culprit? $QEMU_VXHS_PROG is empty for me, which means `set_prog_path
> qnio_server` found nothing to use. You'll have to add in a safety valve
> that only calls 'type' if operating on a non-empty path in the first place.
Sure, will drop the patches and send a v5.
Thanks!
Max
signature.asc
Description: OpenPGP digital signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-block] [PATCH v4 1/2] iotests: Use absolute paths for executables,
Max Reitz <=