Replaces self.alltests, self.changes, self.problems, self.regressions,
self.fixes, and self.skipped with self.test, a dictionary containing all
of the same information. This cleans the namespace for the NewSummary
object, as well as allowing for some other code simplification
Signed-off-by: Dylan
There were whitespace errors as a result of the recent tabs to spaces
conversion. This patch corrects those
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
piglit-summary-junit.py | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/piglit-summary-junit.py
This new page is able to convey the same information, but takes less
time to generate
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
framework/summary.py| 21 -
templates/empty_status.mako | 27 +++
2 files changed, 43 insertions(+), 5
Splits the code that finds problems, regressions, fixes, changes, and
skips out of the constructor. Instead of incurring the overhead of
calculating these every time whether they are needed or not the code can
now be smarter about it's path, only generating the ones it needs, if
any.
Using mako as an alternative to the junit.py file has a couple of
advantages: First there is a lot less code involved, mako just does most
of the work for us. Second, it works in almost the same way as the HTML
generation.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
This method is about 50% faster than the existing implementation which
uses junit.py.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
framework/summary.py | 55
1 file changed, 55 insertions(+)
diff --git a/framework/summary.py
Stop using the old JUnit generation code, and start using the new code.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
piglit-summary-junit.py | 84 +++--
1 file changed, 4 insertions(+), 80 deletions(-)
diff --git a/piglit-summary-junit.py
This field is required by the JUnit xsd, and was not provided. With this
change the JUnit xml being produced is valid.
xsd source: https://svn.jenkins-ci.org/trunk/hudson/dtkit/dtkit-format/
dtkit-junit-model/src/main/resources/com/thalesgroup/dtkit/junit/model/
xsd/junit-4.xsd
Signed-off-by:
Argparse requires less code, and is much easier to read and maintain.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
piglit-summary.py | 94 +--
1 file changed, 29 insertions(+), 65 deletions(-)
diff --git a/piglit-summary.py
Since -s/--summary will always override -d/--diff don't allow them to be
called together. While this may seem a little confusing, the idea of
asking for --diffs and then not getting them is a bug in the original
implementation.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
Since nothing now depends on the old summary backend, delete it. This
first eliminates dead code, but also ensures that no new functionality
becomes dependent on this code.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
framework/summary.py | 232
The old name existed simply as a holdover until the Summary name could
be cleared up. Since it is now available, use the Summary name.
Signed-off-by: Dylan Baker baker.dyla...@gmail.com
---
framework/summary.py| 4 ++--
piglit-summary-html.py | 2 +-
piglit-summary-junit.py | 2 +-
On 10 June 2013 16:54, Eric Anholt e...@anholt.net wrote:
Fabian Bieler fabianbie...@fastmail.fm writes:
Add primitives with adjacency and (for extreme future-proofness) patches
to
recognized drawing modes.
---
tests/shaders/shader_runner.c | 7 ++-
1 file changed, 6
On 3 June 2013 05:39, Fabian Bieler fabianbie...@fastmail.fm wrote:
This is usefull for core profile OpenGL contexts and geometry shaders.
s/usefull/useful/
With that fixed,
Reviewed-by: Paul Berry stereotype...@gmail.com
---
tests/util/piglit-util-gl.c | 22 +++---
1
On 3 June 2013 05:42, Fabian Bieler fabianbie...@fastmail.fm wrote:
This is usefull for core profile OpenGL contexts and geometry shaders.
s/usefull/useful/ here too.
Reviewed-by: Paul Berry stereotype...@gmail.com
---
tests/shaders/shader_runner.c | 6 +++---
1 file changed, 3
On 7 June 2013 11:12, Fabian Bieler fabianbie...@fastmail.fm wrote:
Test that glProgramParameter() triggers the required errors.
The geometry shader input primitive type is limited to a subset
of OpenGL primitive types.
From the ARB_geometry_shader4 spec (section Errors):
The error
Often, I make changes to Mesa which only affect tests that actually
execute on the GPU. For such changes, there's no real benefit to
running compiler-only/front-end tests, such as glslparsertest and
asmparsertest files. There are a lot of those, so cutting them can save
quite a bit of time when
It's probably noting somewhere that these tests will fail on CL
implementations where the maximum local workgroup size is 16 (e.g.
CPU implementations that limit to 1 thread per core).
Also, I'm not sure if it really matters, but instead of:
index2 = index + 1;
index2 = index2 == 4 ? 0 : index2
18 matches
Mail list logo