[Mesa-dev] [PATCH v2] configure+mesa/st: check -std=c++11 support and enable tests accordingly

Emil Velikov emil.l.velikov at gmail.com
Mon Sep 18 15:07:47 UTC 2017


Hi Gert,

On 17 September 2017 at 08:32, Gert Wollny <gw.fossdev at gmail.com> wrote:
> The tests in mesa/st/tests require C++11. Check whether the compiler
> supports the -std=c++11 flag and if so, enable the tests. Otherwise
> skip compiling and running the tests altogether.
>
> Fixes: 7be6d8fe12  ("mesa/st: glsl_to_tgsi: add tests for the
> new temporary lifetime tracker")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102665
> ---
> - Submitter has no mesa-git write access
> - Changes w.r.t v1:
>   * correct white space errors
>   * Add "Fixes" to commit message
>
Adding an extra configure check for a single test is a bit annoying.
Can I ask you to give my earlier suggestion a try?

Namely:
Replace the c++11 initialises with something along the lines of
 "s/const vector<LocalClass>/const LocalClass[]/"

Thanks
Emil


More information about the mesa-dev mailing list