<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Nov 11, 2017 at 8:11 AM, Dylan Baker <span dir="ltr"><<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I sent a patch to fix this as did Rob, I think his landed already.<br></blockquote><div><br></div><div>Yeah...  I still kind-of like the way my patch works.  I personally think it's way nicer to predicate build targets on boolean configure flags rather than setting build_by_default to true and then conditionally including the file.  Maybe that's a silly style choice, but I really like it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not sure it's a good idea to build tools by default. Personally I'd rather see an extra option added if compiling via ninja src/gallium/drivers/freedreno/<wbr>ir3compiler is too tedious.<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>I'm kind-of thinking we want an install-dev-tools flag which will cause all dev tools to build and install if you're already building the relevant driver.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Dylan<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On November 10, 2017 11:10:37 PM PST, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
>Setting build_by_default to true makes it suddenly pull in freedreno<br>
>and<br>
>all of gallium unconditionally.<br>
><br>
>Cc: Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>><br>
>Cc: Dylan Baker <<a href="mailto:dylan@pnwbakers.com">dylan@pnwbakers.com</a>><br>
>---<br>
> src/gallium/drivers/freedreno/<wbr>meson.build | 2 +-<br>
> 1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
>diff --git a/src/gallium/drivers/<wbr>freedreno/meson.build<br>
>b/src/gallium/drivers/<wbr>freedreno/meson.build<br>
>index 3fb94ed..b3a33da 100644<br>
>--- a/src/gallium/drivers/<wbr>freedreno/meson.build<br>
>+++ b/src/gallium/drivers/<wbr>freedreno/meson.build<br>
>@@ -224,5 +224,5 @@ ir3_compiler = executable(<br>
>     libglsl_standalone,<br>
>     libmesa_util,<br>
>   ],<br>
>-  build_by_default : true,<br>
>+  build_by_default : with_gallium_freedreno,<br>
> )<br>
</div></div></blockquote></div><br></div></div>