<div dir="ltr">Hi,<div>I tried to add <span style="font-size:12.8px">LOCAL_GENERATED_SOURCES += $(MESA_DRI_OPTIONS_H)</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">incrementally three places:</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">src/gallium/drivers/radeonsi/Android.mk</span><br></div><div><span style="font-size:12.8px">src/gallium/targets/dri/Android.mk</span><br></div><div><span style="font-size:12.8px">and src/gallium/auxiliary/Android.mk</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">but it didn't work as expected</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">I also tried by putting later the export in src/util/Android.mk</span></div><div><span style="font-size:12.8px">after local generation rules for </span><span style="font-size:12.8px"> </span><span style="font-size:12.8px">$(MESA_DRI_OPTIONS_H).</span></div><div><span style="font-size:12.8px">but it has no effect.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">BTW the include is present in automake rules [1] </span></div><div><span style="font-size:12.8px">as part of </span><span style="font-size:12.8px">GALLIUM_TARGET_CFLAGS [2]</span></div><div><span style="font-size:12.8px">which includes -I$(top_builddir)/src/util/</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">so the patch is at least in conformance with automake rules for gallium_dri target.</span></div><div><span style="font-size:12.8px">If you see better ways please send me istructions</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Mauro</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">[1] <a href="https://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/targets/dri/Makefile.am">https://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/targets/dri/Makefile.am</a></span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">[2]  <a href="https://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/Automake.inc">https://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/Automake.inc</a></span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px"><br></span></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-12-19 23:38 GMT+01:00 Rob Herring <span dir="ltr"><<a href="mailto:robh@kernel.org" target="_blank">robh@kernel.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Dec 18, 2017 at 8:45 PM, Yu, Qiang <<a href="mailto:Qiang.Yu@amd.com">Qiang.Yu@amd.com</a>> wrote:<br>
> I met this problem when upgrade mesa 17.2 to 17.3, and build without<br>
> a make clean. A clean build won't have this problem.<br>
<br>
</span>I think I'm seeing a similar problem on CI builds with current master,<br>
but with intel_screen.c:<br>
<br>
external/mesa3d/src/mesa/<wbr>drivers/dri/i965/intel_screen.<wbr>c:60:7: error:<br>
expected '}'<br>
      DRI_CONF_MESA_NO_ERROR("false"<wbr>)<br>
      ^<br>
<br>
For the CI jobs, it is dependent on which build machine the job runs on.<br>
<br>
<br>
In the past, we've had some cross module dependency problems with<br>
generated files which were fixed with:<br>
<br>
LOCAL_GENERATED_SOURCES += $(MESA_DRI_OPTIONS_H)<br>
<br>
I think this is needed because we need a dependency on the source<br>
file, not just the include directory. libmesa_pipe_radeonsi is missing<br>
this. However, with the problem I'm hitting, this dependency is there.<br>
<span class="HOEnZb"><font color="#888888"><br>
Rob<br>
</font></span></blockquote></div><br></div>