[Piglit] [PATCHv2 00/10] add infrastructure for GLES1 tests
Chia-I Wu
olvaffe at gmail.com
Sat Sep 10 23:21:13 PDT 2011
On Sat, Sep 10, 2011 at 2:34 AM, Paul Berry <stereotype441 at gmail.com> wrote:
> On 8 September 2011 17:52, Chia-I Wu <olvaffe at gmail.com> wrote:
>>
>> On Fri, Sep 9, 2011 at 4:23 AM, Paul Berry <stereotype441 at gmail.com>
>> wrote:
>> > On 5 September 2011 11:51, Ian Romanick <idr at freedesktop.org> wrote:
>> >>
>> >> -----BEGIN PGP SIGNED MESSAGE-----
>> >> Hash: SHA1
>> >>
>> >> On 09/04/2011 07:44 PM, Chia-I Wu wrote:
>> >> > Changes since v1 are
>> >> >
>> >> > - change how switch cases are guarded in piglit_get_gl_error_name
>> >> > - rename piglit-util-gles2.c to piglit-util-gles.c, and let GLES1
>> >> > and
>> >> > GLES2 share it
>> >> > - hide enum naming differences in gl_wrap.h
>> >> > (e.g. define GL_TEXTURE_CUBE_MAP_POSITIVE_X to
>> >> > GL_TEXTURE_CUBE_MAP_POSITIVE_X_OES for GLES1)
>> >> > - call piglit_report_result(PIGLIT_SKIP) when piglit_require_GLSL or
>> >> > other shader related helpers are called in piglitutil_gles1
>> >>
>> >> For the parts of the series that weren't already,
>> >>
>> >> Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
>> >>
>> >> -----BEGIN PGP SIGNATURE-----
>> >> Version: GnuPG v1.4.11 (GNU/Linux)
>> >> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>> >>
>> >> iEUEARECAAYFAk5lGjYACgkQX1gOwKyEAw/+3ACYgihsLK9gGgCUlhhdfnv7jzNR
>> >> UACdH9sy+IJ9R2gHQjxXjLVedNok3y4=
>> >> =BpTb
>> >> -----END PGP SIGNATURE-----
>> >> _______________________________________________
>> >> Piglit mailing list
>> >> Piglit at lists.freedesktop.org
>> >> http://lists.freedesktop.org/mailman/listinfo/piglit
>> >
>> > Chia-I: I was about to push these patches on your behalf, but I'm not
>> > 100%
>> > sure which versions are the right ones to push. I believe you should
>> > have
>> > push access to Piglit now--can you push these patches? Thanks.
>> The ones marked [PATCHv2 ..] should be pushed. I do not have push
>> access to piglit. I should apply after I've created enough patches.
>>
>> --
>> olv at LunarG.com
>
> Ok, patches have been pushed.
>
> FWIW, I think you have enough patches under your belt to get Piglit access,
> so feel free to apply. (In fact, I was talking to Eric yesterday and he
> thought you already had access).
Great, thanks for the help! Now it is me to create more tests :)
--
olv at LunarG.com
More information about the Piglit
mailing list