[Piglit] [PATCH 3/4] utils: add initial WAFFLE_WGL support
Jose Fonseca
jfonseca at vmware.com
Mon Dec 15 04:18:23 PST 2014
On 14/12/14 15:48, Emil Velikov wrote:
> On 12/12/14 15:00, Jose Fonseca wrote:
>> On 23/11/14 15:09, Emil Velikov wrote:
>>> On 23/11/14 09:38, Jose Fonseca wrote:
>>>> On 21/11/14 19:52, Emil Velikov wrote:
>>> [snip]
>>>>> --- /dev/null
>>>>> +++ b/tests/util/piglit-framework-gl/piglit_wgl_framework.c
>>> [snip]
>>>>> +static void
>>>>> +enter_event_loop(struct piglit_winsys_framework *winsys_fw)
>>>>> +{
>>>>> +
>>>>> + /* FINISHME: Write event loop for Windows.
>>>>> + *
>>>>> + * Until we have proper Windows/WGL support, give the user enough
>>>>> + * time to view the window by sleeping.
>>>>> + */
>>>>> + Sleep(8000);
>>>>
>>>> Instead of Sleep for 8 secs, you could do
>>>>
>>>> fprintf(stderr, "Press any key when done.\n");
>>>> getchar().
>>>>
>>>> It would be more practical, and self-documenting.
>>>>
>>> Makes sense. Unless there are other comments I'll amend before pushing.
>>> The latter of which will happen after waffle 1.5.0 is rolled out.
>>>
>>> Thanks
>>> Emil
>>>
>>
>> Emil,
>>
>> Feel free to push this series. Or let me know if you rather I do it.
>>
>> I actually have a couple of follow-on patches to get this working with
>> Mingw better:
>>
>> https://urldefense.proofpoint.com/v2/url?u=http-3A__cgit.freedesktop.org_-7Ejrfonseca_piglit_log_-3Fh-3Dwgl-2Dwaffle&d=AAIDaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=zfmBZnnVGHeYde45pMKNnVyzeaZbdIqVLprmZCM2zzE&m=hlGMP0n52wcvFoFYI11nH3-NtL0Uk3qmcS61N4EhoEk&s=9J-DdsZQet3WgtN_0vzQm8mbICcb8wcnZRUSyTflXMU&e=
>>
> Will push it out in a second. I wanted to have a few days between the
> waffle 1.5.0 release and bumping the minimum requirement in piglit :)
>
> I'm not too trilled with the Mingw fix - things should work like a charm
> without it. What are the problems that you're facing without it ?
They are totally broken for me:
- pkg-config doesn't work for cross compiling: it will find the host's
waffle, instead of the target's waffle (and refuse to build unless the
host's waffle is not found or has an older version)
- I haven't tried to build with MinGW on Windows natively, but
pkg-config doesn't exist on Windows AFAIK, so I doubt it works at all there
Besides that the patch also made things easier for those building this
on Windows.
I splitted that patch into smaller changes, and provided better
explanations. I hope it makes more sense now.
Jose
More information about the Piglit
mailing list