[Mesa-dev] [PATCH mesa v2] glx: add GLXdispatchIndex sort check

Emil Velikov emil.l.velikov at gmail.com
Thu Feb 23 16:07:36 UTC 2017


You're a star Eric, thank you !

On 22 February 2017 at 11:24, Eric Engestrom <eric.engestrom at imgtec.com> wrote:
> Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
> ---
> v2: make sure the list is in the order C's strcmp uses (Ilia)
>     Ilia: I used LC_ALL instead of LANG, as it takes precedence
>           (ie. LANG=C in this script would be overridden by
>           LC_ALL=en_US in the environment).
Personally I'm safe even without the LC_ALL bit, but it seems like a good idea.

Perhaps we want that for the EGL test as well ?
In both cases we really want to set srcdir, such that ./dispatch-index-check and
../some/crazy/path/to/dispatch-index-check also work.

Perhaps reuse the autogen.sh magic ?

Feel free to do that as follow-up or let me know otherwise and I'll
fix it. I'm just fixing our "broken since day 1" glsl tests ;-)

As-is for this patch and the EGL series:
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

-Emil


More information about the mesa-dev mailing list