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 <[email protected]> wrote:

> On 09/08/2015 08:41 AM, Ilia Mirkin wrote:
>
>> On Tue, Sep 8, 2015 at 10:32 AM, Brian Paul <[email protected]> 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
>
>
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to