On 10/01/2012 03:47 PM, Jordan Justen wrote: > Chad, > > Does this look close to the correct way to add GBM support > following your recent piglit framework refactor series? > > -Jordan > > -- > > Use gbm for testing if the PIGLIT_PLATFORM environment variable > is set to gbm.
This patch looks correct to me. Singe piglit_gbm_framework doesn't use include any gbm headers or link to libgbm, then it makes sense to enable PIGLIT_HAS_GBM unconditionally on Linux for now. Reviewed-by: Chad Versace <chad.vers...@linux.intel.com> _______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit