Commit 23e06ffaa51b9075c1e5e0c220c991efc5609838 broke skip handling by
overwriting the result after we had set it.
---
tests/xts.py | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/tests/xts.py b/tests/xts.py
index 6798e70..e528626 100644
--- a/tests/xts.py
+++ b/te
On Wed, 2016-01-27 at 18:47 -0500, Ilia Mirkin wrote:
> Reviwed-by: Ilia Mirkin
>
> Will push this shortly.
>
> On Wed, Jan 27, 2016 at 1:46 PM, Ryan Houdek > wrote:
> > fbo-extended-blend-pattern was using an implicit integer to float
> > conversion
> > which isn't allowed in ESSL.
It would a
From: Dylan Baker
In python 2 < 2.6, generators define a next() method. In python 2.6+
(including 2.7) they should define a __next__ method and are called with
the next() function.
In python 2.6 and 2.7 both are supported, but in 3.x only the __next__
method is supported
Signed-off-by: Dylan Ba
From: Dylan Baker
Python 3 doesn't allow mixing of tabs and spaces for indent, only a
choice between one or the other. Piglit uses spaces.
Signed-off-by: Dylan Baker
---
tests/util/gen_dispatch.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/util/gen_dispatch.py b/
From: Dylan Baker
Signed-off-by: Dylan Baker
---
unittests/base_tests.py| 43 +++
unittests/results_tests.py | 5 +
unittests/utils.py | 5 +
3 files changed, 53 insertions(+)
diff --git a/unittests/base_tests.py b/unittests/base_te
From: Dylan Baker
Signed-off-by: Dylan Baker
---
framework/backends/json.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/framework/backends/json.py b/framework/backends/json.py
index beae8f8..c4074fd 100644
--- a/framework/backends/json.py
+++ b/framework/backends/json.py
From: Dylan Baker
Convert to unicode, this requires reworking the tests to ensure that
they provide bytes instances in the right places, or they trip up on the
decode method call, which in python 3 is only available on str
(unicode), and not on bytes.
Signed-off-by: Dylan Baker
---
framework/t
From: Dylan Baker
Signed-off-by: Dylan Baker
---
framework/results.py | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/framework/results.py b/framework/results.py
index d8a9f16..f0c0a99 100644
--- a/framework/results.py
+++ b/framework/results.py
@@ -29,7 +29,7 @@ import
From: Dylan Baker
For some reason in python 3 (all versions) some of the mock patchers in
this opengl test leak out into other modules. This doesn't happen in
python 2. I'm not sure why this fixes the issue, but it does.
Signed-off-by: Dylan Baker
---
unittests/opengl_tests.py | 26 +++
From: Dylan Baker
This changes collect_system_info to convert the raw output of each
command to unicode before returning it.
Signed-off-by: Dylan Baker
---
framework/core.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/framework/core.py b/framework/core.py
index 5ef7
From: Dylan Baker
In python 3 the current implementation provides bytes instead of
unicode, but the write mode is set for unicode.
Signed-off-by: Dylan Baker
---
framework/summary/html_.py | 2 --
1 file changed, 2 deletions(-)
diff --git a/framework/summary/html_.py b/framework/summary/html_
From: Dylan Baker
This refactors the tox.ini a little bit since some of the requirements
of the accel platform on 2.7 (backports.lzma and subprocess32) are not
required in 3.3+, and mock is not required on 3.3+ for any profile.
Signed-off-by: Dylan Baker
---
tox.ini | 10 +-
1 file cha
From: Dylan Baker
This makes the GLSLParserTest class only support str (unicode in python
2), and not bytes (str in python 2).
Signed-off-by: Dylan Baker
---
framework/test/glsl_parser_test.py | 27 +++
framework/test/shader_test.py | 9 ++---
unittests/glsl
From: Dylan Baker
On python 2 re.compile sets no flags, so asking if the flags bitfield ==
re.IGNORECASE works. For python 3 this isn't the case (the default it
returns for me is 100010), so the test needs to correctly check that
re.IGNORECASE is in the bitfield.
Signed-off-by: Dylan Baker
---
From: Dylan Baker
These tests are always valid on python 3, but only valid on python 2 if
subprocess32 is installed.
One unittest on python3 raises a warning. I've tracked it back to somewhere
in the python stdlib. I think that for some reason a file is being
unlinked twice, but I'm having troub
From: Dylan Baker
This actually needs to handle bytes, since it gets assigned from
subprocess directly.
Signed-off-by: Dylan Baker
---
unittests/results_tests.py | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/unittests/results_tests.py b/unittests/results_te
From: Dylan Baker
This changes the junit.py script to always convert bytes into unicode
before writing, since we do declare that our xml is encoded in unicode.
Signed-off-by: Dylan Baker
---
framework/backends/junit.py | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --gi
From: Dylan Baker
In python2 one can seek to negatives (-1 is the end of the file, for
example). In python 2.6+ (and exclusively in python 3) one instead uses
the second argument of seek to say whether to start at the start or end
of the file, and then an exact point. So file.seek(-1) -> file.see
From: Dylan Baker
The generators have supported 3.x for some time, this now makes 3.x the
default, and falls back to 2.7
This has been tested with 3.5 and 2.7
Signed-off-by: Dylan Baker
---
CMakeLists.txt | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/CMakeLists.txt b/
From: Dylan Baker
By default the builtins don't appear in the python 3 namespace, and
create=True must be set to make this work.
Signed-off-by: Dylan Baker
---
unittests/options_tests.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/unittests/options_tests.py b/unittest
From: Dylan Baker
As of python 3.3 mock is included in python's stdlib. Try to import the
included version, then the external package version if that fails.
This patch was generated with the following sed command:
sed -i -e 's@import mock@try:\nfrom unittest import mock\nexcept
ImportError:
From: Dylan Baker
Mostly this involves marking str (unicode) and bytes explicitly and not
mixing the two.
Signed-off-by: Dylan Baker
---
framework/dmesg.py | 3 ++-
unittests/dmesg_tests.py | 19 ++-
2 files changed, 12 insertions(+), 10 deletions(-)
diff --git a/framew
From: Dylan Baker
This makes metaclasses work on both python 3 and python 2, which have a
different syntax for setting the metaclass of a class.
Signed-off-by: Dylan Baker
---
framework/backends/abstract.py | 5 +++--
framework/dmesg.py | 5 +++--
framework/log.py | 5
From: Dylan Baker
Instead allow extra arguments to be passed to tox, which can be used to
set the coverage options if wanted.
Signed-off-by: Dylan Baker
---
tox.ini | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tox.ini b/tox.ini
index 0721417..1914d1e 100644
--- a/tox
From: Dylan Baker
This makes the piglit executable run with python 3 instead of python 2,
the legacy scripts (piglit-run, piglit-summary-html, etc) still run with
python 2.
The goal of this series has been to make python 2.7 a fallback rather
than a main option. If one wants to use python 2, it'
From: Dylan Baker
Signed-off-by: Dylan Baker
---
unittests/results_tests.py | 10 ++
1 file changed, 10 insertions(+)
diff --git a/unittests/results_tests.py b/unittests/results_tests.py
index 6870864..dc004d5 100644
--- a/unittests/results_tests.py
+++ b/unittests/results_tests.py
@@
From: Dylan Baker
There wasn't ever a need to use a comprehension here, this is simpler
and easier to read and maintain. It might even be faster.
Signed-off-by: Dylan Baker
---
framework/profile.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/framework/profile.py b/f
From: Dylan Baker
These are the three python3 like behaviors that piglit should rely on.
The only other applicable future import is unicode_literals. Although my
plan is to use unicode_literals, that will actually cause behavioral
changes in some cases, where these cause minimal changes to the co
From: Dylan Baker
This is itertools.izip on python 2, and builtins.zip in python 3. Which
are the same lazy function.
Signed-off-by: Dylan Baker
---
framework/grouptools.py | 2 ++
framework/summary/common.py | 5 +++--
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/framewo
From: Dylan Baker
Signed-off-by: Dylan Baker
---
unittests/status_tests.py | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/unittests/status_tests.py b/unittests/status_tests.py
index b76d5b5..00f00e8 100644
--- a/unittests/status_tests.py
+++ b/unittests/status_tests.
From: Dylan Baker
This is xrange on python 2.x and range on 3.x
Signed-off-by: Dylan Baker
---
framework/test/base.py| 4 +++-
tests/all.py | 4 +++-
unittests/summary_common_tests.py | 3 ++-
3 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/frame
From: Dylan Baker
Apparently these aren't inherited in python 3, which makes sense.
Signed-off-by: Dylan Baker
---
framework/status.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/framework/status.py b/framework/status.py
index 4c8b421..7fca856 100644
--- a/framework/status.py
+++ b/
From: Dylan Baker
Signed-off-by: Dylan Baker
---
unittests/core_tests.py | 7 ---
unittests/summary_html_tests.py | 3 ++-
unittests/utils.py | 3 ++-
3 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/unittests/core_tests.py b/unittests/core_tests.py
index
From: Dylan Baker
This module is a compatibility shim for python 3.x and 2.x
Signed-off-by: Dylan Baker
---
framework/core.py | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/framework/core.py b/framework/core.py
index bcf4ea5..fbe7377 100644
--- a/framework/
From: Dylan Baker
In python 2 the write mode should be 'w', and the read 'r', but in
python 3 they need to be 'wt', and 'rt' respectively. Python 3 also has
an lzma module built in, which vastly simplifies supporting xz.
Signed-off-by: Dylan Baker
---
framework/backends/compression.py | 21
From: Dylan Baker
Use unicode_literals from __future__. This makes undecorated strings
(those not using and b or u prefix) into unicode instead of bytes in
python 2. This means that bytes strings need to have a b prefix now.
This also fixes a couple of unittests that broke during the transition.
From: Dylan Baker
All of these changes are submitted to upstream six, and are ontrack for
landing.
Signed-off-by: Dylan Baker
---
framework/compat.py | 41 +
1 file changed, 41 insertions(+)
create mode 100644 framework/compat.py
diff --git a/framework
From: Dylan Baker
This implements the same behavior, but with a fallback parameter.
Signed-off-by: Dylan Baker
---
framework/programs/run.py | 31 ---
1 file changed, 12 insertions(+), 19 deletions(-)
diff --git a/framework/programs/run.py b/framework/programs/run.
From: Dylan Baker
This is not feature complete for python 3.x in and of itself, but it
gets started by using six functions rather than str and unicode.
Signed-off-by: Dylan Baker
---
framework/backends/compression.py | 8 +---
framework/backends/junit.py | 2 +-
framework/exception
From: Dylan Baker
This wraps the subprocess32/not subprocess32 in a six.PY2 block, and for
python 3.x just imports subprocess and sets _EXTRA_ARGS appropriately.
For MS Windows the timeout is disabled. This could be fixed, but I don't
have any interest in getting it working.
Signed-off-by: Dyla
From: Dylan Baker
Signed-off-by: Dylan Baker
---
framework/test/base.py | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/framework/test/base.py b/framework/test/base.py
index 43f81c4..0c15808 100644
--- a/framework/test/base.py
+++ b/framework/test/base.py
@@ -129,8 +129,7
From: Dylan Baker
Also fix unittest that relied on compare to None having python 2
behavior.
Signed-off-by: Dylan Baker
---
framework/test/base.py| 5 +
unittests/oglconform_tests.py | 1 +
2 files changed, 6 insertions(+)
diff --git a/framework/test/base.py b/framework/test/base.
From: Dylan Baker
This is the fast C based version on python 2, and is adapative on
python3, importing the C based version if it's available and falling
back if necessary.
Signed-off-by: Dylan Baker
---
framework/backends/compression.py | 10 --
unittests/oglconform_tests.py | 6
From: Dylan Baker
Signed-off-by: Dylan Baker
---
framework/log.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/framework/log.py b/framework/log.py
index 5bd6902..704ab05 100644
--- a/framework/log.py
+++ b/framework/log.py
@@ -32,13 +32,14 @@ import abc
import itertool
From: Dylan Baker
This rather lengthy series incrementally ports the framework from
running only on python 2.7 to running on python 2.7 and 3.3+ (3.2 might
already work, but I haven't tested it. If there is interest in 3.2
support I'd rather worry about that after landing this series).
This appr
From: Dylan Baker
This was initially generated via the following sed command:
sed -i \
-e 's@in \(.*\)\.iter\(values,keys,items\)()@in six.iter\2(\1)@g' \
-e 's@in \(.*\..*\)\.iter\(values,keys,items\)()@in six.iter\2(\1)@g'
Then cleaned up by hand, including changes for view*.
Signed-o
From: Dylan Baker
By default in python 3 write modes for tempfile are bytes, but piglit
uses unicode. This fixes that.
Signed-off-by: Dylan Baker
---
unittests/utils.py | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/unittests/utils.py b/unittests/utils.py
index f383
Reviwed-by: Ilia Mirkin
Will push this shortly.
On Wed, Jan 27, 2016 at 1:46 PM, Ryan Houdek wrote:
> fbo-extended-blend-pattern was using an implicit integer to float conversion
> which isn't allowed in ESSL.
>
> builtins program linker error checking was dumb, do explicit link checking
> ins
On Fri, Jan 15, 2016 at 05:41:08PM -0500, Jan Vesely wrote:
> Signed-off-by: Jan Vesely
> ---
>
> This used to trigger failure when LLVM put different types in different
> .rodata sections. Now there are no .rodata sections for radeon, but I think
> we should still test it.
>
> Jan
>
Reviewe
On Fri, Jan 15, 2016 at 05:38:43PM -0500, Jan Vesely wrote:
> Deprecated features are required to be supported until they are removed.
>
> Signed-off-by: Jan Vesely
Reviewed-by: Tom Stellard
> ---
> tests/cl/api/get-image-info.c | 14 --
> 1 file changed, 4 insertions(+), 10 deletio
On Fri, Jan 15, 2016 at 05:37:42PM -0500, Jan Vesely wrote:
> Signed-off-by: Jan Vesely
Reviewed-by: Tom Stellard
> ---
> tests/cl/program/execute/constant-load.cl | 18 ++
> 1 file changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/tests/cl/program/execute/constant-lo
https://bugs.freedesktop.org/show_bug.cgi?id=93856
--- Comment #1 from Dylan ---
Hmmm, I can't replicate this myself (I'm testing with dEQP and piglit opengl,
because I don't really want to have to reboot my system when igt hoses it). You
are running an older version of piglit, this may have been
fbo-extended-blend-pattern was using an implicit integer to float conversion
which isn't allowed in ESSL.
builtins program linker error checking was dumb, do explicit link checking
instead.
---
tests/spec/arb_blend_func_extended/api/builtins.c| 5 -
.../arb_blend_func_ext
53 matches
Mail list logo