Re: [PATCH v5 2/5] iotests: add script_initialize

[Max Reitz]

On 23.09.19 15:30, Max Reitz wrote:
> On 18.09.19 01:45, John Snow wrote:
>> Like script_main, but doesn't require a single point of entry.
>> Replace all existing initialization sections with this drop-in replacement.
>>
>> This brings debug support to all existing script-style iotests.
>>
>> Any specification for supported_oses=['linux'] was dropped as explained
>> in the previous commit, because there was never any reason to limit python
>> tests to linux-only.
>>
>> Signed-off-by: John Snow <address@hidden>
>> ---
>>  tests/qemu-iotests/149        |  3 +-
>>  tests/qemu-iotests/194        |  3 +-
>>  tests/qemu-iotests/202        |  3 +-
>>  tests/qemu-iotests/203        |  3 +-
>>  tests/qemu-iotests/206        |  2 +-
>>  tests/qemu-iotests/207        |  2 +-
>>  tests/qemu-iotests/208        |  2 +-
>>  tests/qemu-iotests/209        |  2 +-
>>  tests/qemu-iotests/210        |  2 +-
>>  tests/qemu-iotests/211        |  2 +-
>>  tests/qemu-iotests/212        |  2 +-
>>  tests/qemu-iotests/213        |  2 +-
>>  tests/qemu-iotests/216        |  3 +-
>>  tests/qemu-iotests/218        |  2 +-
>>  tests/qemu-iotests/219        |  2 +-
>>  tests/qemu-iotests/222        |  5 ++-
>>  tests/qemu-iotests/224        |  3 +-
>>  tests/qemu-iotests/228        |  3 +-
>>  tests/qemu-iotests/234        |  3 +-
>>  tests/qemu-iotests/235        |  4 +--
>>  tests/qemu-iotests/236        |  2 +-
>>  tests/qemu-iotests/237        |  2 +-
>>  tests/qemu-iotests/238        |  2 ++
>>  tests/qemu-iotests/242        |  2 +-
>>  tests/qemu-iotests/246        |  2 +-
>>  tests/qemu-iotests/248        |  2 +-
>>  tests/qemu-iotests/254        |  2 +-
>>  tests/qemu-iotests/255        |  2 +-
>>  tests/qemu-iotests/256        |  2 +-
>>  tests/qemu-iotests/262        |  3 +-
>>  tests/qemu-iotests/iotests.py | 62 ++++++++++++++++++++++-------------
>>  31 files changed, 73 insertions(+), 63 deletions(-)
> 
> Reviewed-by: Max Reitz <address@hidden>

(Forgot to add: Needs a bit of rebase on Kevin’s patch to drop Python 2
support, but the changes look obvious enough.)

signature.asc
Description: OpenPGP digital signature