|
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
[Prev in Thread] | Current Thread | [Next in Thread] |