[Mesa-dev] [PATCH] Android: gallium_dri: add include to get "xmlpool/options.h"

Rob Herring robh at kernel.org
Tue Dec 19 22:38:00 UTC 2017


On Mon, Dec 18, 2017 at 8:45 PM, Yu, Qiang <Qiang.Yu at amd.com> wrote:
> I met this problem when upgrade mesa 17.2 to 17.3, and build without
> a make clean. A clean build won't have this problem.

I think I'm seeing a similar problem on CI builds with current master,
but with intel_screen.c:

external/mesa3d/src/mesa/drivers/dri/i965/intel_screen.c:60:7: error:
expected '}'
      DRI_CONF_MESA_NO_ERROR("false")
      ^

For the CI jobs, it is dependent on which build machine the job runs on.


In the past, we've had some cross module dependency problems with
generated files which were fixed with:

LOCAL_GENERATED_SOURCES += $(MESA_DRI_OPTIONS_H)

I think this is needed because we need a dependency on the source
file, not just the include directory. libmesa_pipe_radeonsi is missing
this. However, with the problem I'm hitting, this dependency is there.

Rob


More information about the mesa-dev mailing list