[Piglit] [PATCH 0/2] shader_runner: Add stricter requirements for [require] section

Stuart Abercrombie sabercrombie at google.com
Thu Feb 7 13:35:12 PST 2013


Thanks for looking into the problem.

The new version I sent in produced the same number of passes as Piglit with
the revert on Sandy Bridge for all.tests without glean.

Stuart


On Wed, Feb 6, 2013 at 9:18 PM, Chad Versace
<chad.versace at linux.intel.com>wrote:

> On 02/06/2013 06:11 PM, Stuart Abercrombie wrote:
> > Sorry about the breakage.  I need to resign myself to a full test run
> every
> > time.
> >
> > Is part of this message missing?
> >
>
> Strange, it seems that git-send-email choked after patch 0/2.
>
> Since you already sent a remake of the reverted patch, my little 2-patch
> series is moot. There's no reason now for me to send the missing emails.
>
>
> > On Wed, Feb 6, 2013 at 5:37 PM, Chad Versace
> > <chad.versace at linux.intel.com>wrote:
> >
> >> This 2-patch series is based on Stuart's patch, titled
> >>   shader_runner: Require [require] and GLSL version requirement
> >>
> >> I committed Stuart's patch today, and then Eric rightfully reverted it.
> >> This
> >> series shouldn't be buggy like the original patch.
> >>
> >> No regressions on a full run on Sandybridge and mesa master.
> >>
> >> Chad Versace (2):
> >>   shader_runner: Require [require] section
> >>   shader_runner: Require either a GLSL version or GL_ARB_fp extension
> >>
> >>  tests/shaders/shader_runner.c | 24 +++++++++++++++++++++++-
> >>  1 file changed, 23 insertions(+), 1 deletion(-)
> >>
> >> --
> >> 1.8.1.1
> >>
> >>
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130207/11a0accb/attachment.html>


More information about the Piglit mailing list