qemu-block
[Top][All Lists]
Advanced

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

Re: [Qemu-block] [Qemu-devel] [PATCH 01/12] qemu-iotests: remove dead co


From: Paolo Bonzini
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH 01/12] qemu-iotests: remove dead code
Date: Thu, 10 Aug 2017 11:53:56 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1

On 10/08/2017 00:18, Eric Blake wrote:
> On 08/09/2017 04:54 PM, Paolo Bonzini wrote:
>> This includes shell function, shell variables, command line options
>> (randomize.awk does not exist) and conditions that can never be true
>> (./qemu does not exist anymore).
> 
> Can we point to a commit id where we stopped making ./qemu?  Is it still
> worth supporting a local symlink?
> 
>>
>> Signed-off-by: Paolo Bonzini <address@hidden>
>> ---
>>  tests/qemu-iotests/check         | 36 +--------------------
>>  tests/qemu-iotests/common        | 23 --------------
>>  tests/qemu-iotests/common.config | 26 ---------------
>>  tests/qemu-iotests/common.rc     | 68 
>> ----------------------------------------
>>  4 files changed, 1 insertion(+), 152 deletions(-)
>>
>> -#
>> -# - These can be added to $HOST_CONFIG_DIR (witch default to ./config)
> 
> Good riddance to the wrong word ('which' was intended)
> 
> Other than possibly still wanting ./qemu convenience, this looks like a
> reasonable cleanup.

In my reply yesterday, I forgot to note that the other current-directory
searches go away in patch 9 anyway. :)

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]