<div dir="ltr"><div>So, I can confirm that with this patch applied to the rc3 tag, I can generate a tarball on Arch with make dist, and then use said tarball to build with SWR enabled on fedora.  So:<br><br></div>Tested-by: Chuck Atkins <<a href="mailto:chuck.atkins@kitware.com">chuck.atkins@kitware.com</a>></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">- Chuck<br></div></div></div>
<br><div class="gmail_quote">On Tue, Jun 14, 2016 at 1:14 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 14 June 2016 at 18:06, Rowley, Timothy O <<a href="mailto:timothy.o.rowley@intel.com">timothy.o.rowley@intel.com</a>> wrote:<br>
><br>
>> On Jun 13, 2016, at 8:03 PM, Rowley, Timothy O <<a href="mailto:timothy.o.rowley@intel.com">timothy.o.rowley@intel.com</a>> wrote:<br>
>><br>
>> A clean tree build works with this version, but distcheck fails:<br>
>><br>
>> ...<br>
>> rm -f config.status config.cache config.log configure.lineno config.status.lineno<br>
>> rm -f Makefile<br>
>> ERROR: files left in build directory after distclean:<br>
>> ./src/gallium/drivers/swr/rasterizer/jitter/builder_gen.cpp<br>
>> ./src/gallium/drivers/swr/rasterizer/jitter/builder_x86.cpp<br>
>> ./src/gallium/drivers/swr/rasterizer/jitter/builder_gen.h<br>
>> make[1]: *** [distcleancheck] Error 1<br>
>> make[1]: Leaving directory `/home/torowley/work/mesa-opt/mesa-12.1.0-devel/_build'<br>
>> make: *** [distcheck] Error 1<br>
>><br>
>> Not sure how builder_x86.cpp managed to change its status.<br>
><br>
> To answer my own question: the reason for builder_x86.cpp being regenerated is because of its dependency on builder_gen.h (through builder.h).<br>
><br>
</span>I thought that one was mentioned is the big comment in the patch.<br>
Perhaps my wording could be improved - any suggestions ?<br>
<br>
And yes, due to the missing dependency the file will be (re)generated<br>
at a later stage thus we'll need to add yet another workaround for<br>
that. Just listing the whole lot in CLEANFILES should be enough.<br>
<br>
Feel free to give it a try, or I'll do at some point later on today.<br>
<br>
Thanks,<br>
Emil<br>
</blockquote></div><br></div>