Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests

2017-10-04 Thread Kevin Wolf
Am 12.09.2017 um 16:44 hat Paolo Bonzini geschrieben: > The purpose of this series is to separate the "check" sources from > the tests. After these patches, common.config is reduced to simple > shell initialization, and common.rc is only included by the tests. > > Along the way, a lot of dead cod

Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests

2017-09-13 Thread Eric Blake
On 09/13/2017 05:06 PM, Paolo Bonzini wrote: > On 13/09/2017 09:59, Fam Zheng wrote: >> On Wed, 09/13 08:47, Thomas Huth wrote: >>> Meta comment: Could we maybe also rename "tests/qemu-iotests" to >>> "tests/iotests" ? The "qemu" prefix sounds always very superfluous to me >>> here... >> >> Sounds

Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests

2017-09-13 Thread Paolo Bonzini
On 13/09/2017 09:59, Fam Zheng wrote: > On Wed, 09/13 08:47, Thomas Huth wrote: >> Meta comment: Could we maybe also rename "tests/qemu-iotests" to >> "tests/iotests" ? The "qemu" prefix sounds always very superfluous to me >> here... > > Sounds good, and saves typing for when this path is manuall

Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests

2017-09-13 Thread Fam Zheng
On Wed, 09/13 08:47, Thomas Huth wrote: > Meta comment: Could we maybe also rename "tests/qemu-iotests" to > "tests/iotests" ? The "qemu" prefix sounds always very superfluous to me > here... Sounds good, and saves typing for when this path is manually entered. But maybe keep tests/qemu-iotests as

Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests

2017-09-12 Thread Thomas Huth
On 12.09.2017 16:44, Paolo Bonzini wrote: > The purpose of this series is to separate the "check" sources from > the tests. After these patches, common.config is reduced to simple > shell initialization, and common.rc is only included by the tests. > > Along the way, a lot of dead code is removed

[Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests

2017-09-12 Thread Paolo Bonzini
The purpose of this series is to separate the "check" sources from the tests. After these patches, common.config is reduced to simple shell initialization, and common.rc is only included by the tests. Along the way, a lot of dead code is removed too. In v2, the following patches: qemu-iotests