[Piglit] [PATCH] Porting basic rendering GL tests from Glean to Piglit

[Juliet Fru]

Hello Brian, IIia,


I already commented the new piglit test for the GL rendering test. I will
make recommended changes and send a new patch to the mailing list. I
already ported the next test; waiting for your corrections too.

Thanks,
Juliet

On Tue, Sep 8, 2015 at 4:11 PM, Brian Paul <brianp at vmware.com> wrote:

> On 09/08/2015 08:41 AM, Ilia Mirkin wrote:
>
>> On Tue, Sep 8, 2015 at 10:32 AM, Brian Paul <brianp at vmware.com> wrote:
>>
>>> On 09/08/2015 08:11 AM, Ilia Mirkin wrote:
>>>
>>>>
>>>> This commit should also remove the relevant test from glean (and
>>>> all.py).
>>>>
>>>
>>>
>>> Juliet already posted a patch which removes the glean test.  I'm happy to
>>> keep that as a separate change.
>>>
>>
>> OK, well, then I assume you'll be comparing the two? I'm definitely
>> not spending the time searching around for other emails or figuring
>> out where in glean this came from.
>>
>
> I'm mentoring Julia for her ESOC project so I'm reviewing her work. Also,
> I wrote the original glean test.
>
> Julie, you can mention in the commit message that this new test replaces
> the original glean tpaths.cpp test.
>
> -Brian
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150908/c785a300/attachment.html>