Re: [Piglit] [PATCH] travis: drop python 3.3

2018-07-19 Thread Andres Gomez
On Thu, 2018-07-19 at 22:16 -0400, Rhys Kidd wrote: > On Thu, 19 Jul 2018 at 18:59, Andres Gomez wrote: > > It's quite old and not even supported by wheel any more. > > > > Cc: Juan A. Suarez Romero > > Cc: Dylan Baker > > Signed-off-by: Andres Gomez > > Hello Andres, > > There's a couple of

Re: [Piglit] [PATCH] framework: deepcopy TestProfile's filters when copying it

2018-07-19 Thread Rhys Kidd
On Thu, 19 Jul 2018 at 19:47, Dylan Baker wrote: > Reviewed-by: Dylan Baker > Reviewed-by: Rhys Kidd Eric Engestrom and I had been looking at those regressions, and came to a similar conclusion. > > Quoting Andres Gomez (2018-07-19 15:39:05) > > This avoids modifications in the original fil

Re: [Piglit] [PATCH] travis: drop python 3.3

2018-07-19 Thread Rhys Kidd
On Thu, 19 Jul 2018 at 18:59, Andres Gomez wrote: > It's quite old and not even supported by wheel any more. > > Cc: Juan A. Suarez Romero > Cc: Dylan Baker > Signed-off-by: Andres Gomez > Hello Andres, There's a couple of related python3.3 segments that you'd want to remove concurrently. I

Re: [Piglit] [PATCH] framework: deepcopy TestProfile's filters when copying it

2018-07-19 Thread Dylan Baker
Reviewed-by: Dylan Baker Quoting Andres Gomez (2018-07-19 15:39:05) > This avoids modifications in the original filters object. > > Detected through Travis CI. > > Fixed also a couple of typos. > > Fixes: 1f0f2ad25 ("framework: Add filters class") > Cc: Dylan Baker > Cc: Michel Dänzer > Sign

Re: [Piglit] [PATCH v2] arb_shader_image_load_store: Test format incompatible texture buffer

2018-07-19 Thread Francisco Jerez
Danylo Piliaiev writes: > Test for the regression which happened when GL_TEXTURE_BUFFER was > allowed to have incompatible format. > > v2: Removed unnecessary code duplication - use upload_image instead > of init_level. (Francisco Jerez) > > Bugzilla: https://bugs.freedesktop.org/show_bug.cg

[Piglit] [PATCH] travis: drop python 3.3

2018-07-19 Thread Andres Gomez
It's quite old and not even supported by wheel any more. Cc: Juan A. Suarez Romero Cc: Dylan Baker Signed-off-by: Andres Gomez --- .travis.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index b4cbcca49..b6e8136e8 100644 --- a/.travis.yml +++ b/.travis.yml @@

[Piglit] [PATCH] framework: deepcopy TestProfile's filters when copying it

2018-07-19 Thread Andres Gomez
This avoids modifications in the original filters object. Detected through Travis CI. Fixed also a couple of typos. Fixes: 1f0f2ad25 ("framework: Add filters class") Cc: Dylan Baker Cc: Michel Dänzer Signed-off-by: Andres Gomez --- framework/profile.py| 5 - unitt

Re: [Piglit] [PATCH] tox: remove pytest-warnings and set pytest-timeout to 1.2.1

2018-07-19 Thread Dylan Baker
Reviewed-by: Dylan Baker Quoting Andres Gomez (2018-07-19 09:08:12) > pytest-warnings is not needed since it was integrated into pytest's > core since 3.1 > > pytest-timeout >= 1.3.0 requires pytest >= 3.6.0 and we want to keep > using 3.2.5 > > Cc: Juan A. Suarez Romero > Cc: Dylan Baker > S

[Piglit] [PATCH] tox: remove pytest-warnings and set pytest-timeout to 1.2.1

2018-07-19 Thread Andres Gomez
pytest-warnings is not needed since it was integrated into pytest's core since 3.1 pytest-timeout >= 1.3.0 requires pytest >= 3.6.0 and we want to keep using 3.2.5 Cc: Juan A. Suarez Romero Cc: Dylan Baker Signed-off-by: Andres Gomez --- tox.ini | 3 +-- 1 file changed, 1 insertion(+), 2 dele

[Piglit] [Bug 107270] shader_test.py GLES and GL shaders in the same command!

2018-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107270 Mark Janes changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Piglit] [PATCH v2] arb_shader_image_load_store: Test format incompatible texture buffer

2018-07-19 Thread Danylo Piliaiev
Test for the regression which happened when GL_TEXTURE_BUFFER was allowed to have incompatible format. v2: Removed unnecessary code duplication - use upload_image instead of init_level. (Francisco Jerez) Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106465 Signed-off-by: Danylo Pil

Re: [Piglit] [PATCH] arb_shader_image_load_store: Test format incompatible texture buffer

2018-07-19 Thread Danylo Piliaiev
On 18.07.18 22:08, Francisco Jerez wrote: Danylo Piliaiev writes: On 18.07.18 00:01, Francisco Jerez wrote: Danylo Piliaiev writes: Test for the regression which happened when GL_TEXTURE_BUFFER was allowed to have incompatible format. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?

[Piglit] [Bug 107270] shader_test.py GLES and GL shaders in the same command!

2018-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107270 --- Comment #5 from Sergii Romantsov --- Thanks, Michel. Seems you are right and it was some mess in my repo. Fixed. -- You are receiving this mail because: You are the QA Contact for the bug.___ Pig

[Piglit] [Bug 107270] shader_test.py GLES and GL shaders in the same command!

2018-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107270 --- Comment #4 from Michel Dänzer --- Dylan's patch fixes the problem for me. Note that the bisect result in comment 2 was incorrect (the failure was introduced by commit 359640285258 "EXT_shader_framebuffer_fetch: Add test cases specific to de

Re: [Piglit] [PATCH 2/2] generated_tests: Make API a subdir in gen_shader_framebuffer_fetch_tests

2018-07-19 Thread Michel Dänzer
On 2018-07-18 08:53 PM, Dylan Baker wrote: > This is required to make the shader_runner multiple tests in one process > mode work, as it cannot mix gles and gl shaders together. Both patches are Tested-by: Michel Dänzer Thanks! -- Earthling Michel Dänzer | http://