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