qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] iotests: Test prefix for supported_oses in Pyth


From: Max Reitz
Subject: Re: [Qemu-devel] [PATCH] iotests: Test prefix for supported_oses in Python
Date: Thu, 22 Jan 2015 12:42:30 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0

On 2015-01-22 at 11:52, Max Reitz wrote:
sys.platform may be set to "linux2" instead of just "linux" (in theory,
it could even be set to "linux3" or something else). The Python
documentation on sys.platform recommends testing the prefix of the
platform against "linux", so we should do that.

Signed-off-by: Max Reitz <address@hidden>
---
  tests/qemu-iotests/iotests.py | 8 +++++++-
  1 file changed, 7 insertions(+), 1 deletion(-)

NACK, somehow missed the patch from Fam himself. (And then I didn't use Stefan's block branch but Kevin's, so it was missing there, because Stefan's pull request last week apparently didn't go through)

So it's already fixed and this patch is no longer required. Thanks John for pointing it out to me!

Max



reply via email to

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