Re: [Qemu-devel] [PATCH v2] qemu-iotests: Fix supported_oses check

2015-01-20 Thread Stefan Hajnoczi
On Fri, Jan 16, 2015 at 01:51:53PM -0500, John Snow wrote: > > > On 01/16/2015 08:23 AM, Stefan Hajnoczi wrote: > >On Fri, Jan 16, 2015 at 09:38:42AM +0800, Fam Zheng wrote: > >>There is a bug in the recently added sys.platform test, and we no longer > >>run python tests, because "linux2" is the

Re: [Qemu-devel] [PATCH v2] qemu-iotests: Fix supported_oses check

2015-01-16 Thread John Snow
On 01/16/2015 08:23 AM, Stefan Hajnoczi wrote: On Fri, Jan 16, 2015 at 09:38:42AM +0800, Fam Zheng wrote: There is a bug in the recently added sys.platform test, and we no longer run python tests, because "linux2" is the value to compare here. So do a prefix match. According to python doc [1],

Re: [Qemu-devel] [PATCH v2] qemu-iotests: Fix supported_oses check

2015-01-16 Thread Stefan Hajnoczi
On Fri, Jan 16, 2015 at 09:38:42AM +0800, Fam Zheng wrote: > There is a bug in the recently added sys.platform test, and we no longer > run python tests, because "linux2" is the value to compare here. So do a > prefix match. According to python doc [1], the way to use sys.platform > is "unless you

[Qemu-devel] [PATCH v2] qemu-iotests: Fix supported_oses check

2015-01-15 Thread Fam Zheng
There is a bug in the recently added sys.platform test, and we no longer run python tests, because "linux2" is the value to compare here. So do a prefix match. According to python doc [1], the way to use sys.platform is "unless you want to test for a specific system version, it is therefore recomme