<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 26, 2016 at 2:06 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@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 Wed, Oct 26, 2016 at 2:00 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Oct 26, 2016 12:51 PM, "Matt Turner" <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br>
>><br>
>> ---<br>
>>  src/intel/<a href="http://Makefile.genxml.am" rel="noreferrer" target="_blank">Makefile.genxml.am</a> | 2 +-<br>
>>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
>><br>
>> diff --git a/src/intel/<a href="http://Makefile.genxml.am" rel="noreferrer" target="_blank">Makefile.genxml.am</a> b/src/intel/<a href="http://Makefile.genxml.am" rel="noreferrer" target="_blank">Makefile.genxml.am</a><br>
>> index 2df223a..2ed2741 100644<br>
>> --- a/src/intel/<a href="http://Makefile.genxml.am" rel="noreferrer" target="_blank">Makefile.genxml.am</a><br>
>> +++ b/src/intel/<a href="http://Makefile.genxml.am" rel="noreferrer" target="_blank">Makefile.genxml.am</a><br>
>> @@ -33,7 +33,7 @@ $(GENXML_GENERATED_FILES): genxml/gen_pack_header.py<br>
>><br>
>>  .xml_pack.h:<br>
>>         $(MKDIR_GEN)<br>
>> -       $(PYTHON_GEN) $(srcdir)/genxml/gen_pack_<wbr>header.py $< > $@<br>
>> +       $(PYTHON_GEN) $(srcdir)/genxml/gen_pack_<wbr>header.py $< > $@ ||<br>
>> ($(RM) $@; false)<br>
><br>
> Thanks for fixing this!  I think the way we do this most other places is<br>
> with "> $@.tmp && mv $@.tmp $&” not that I care that much...<br>
<br>
</span>In fact, the opposite is true. We do the .tmp trick with the git sha1<br>
file, but use what's done in this patch in multiple places in<br>
<a href="http://Makefile.glsl.am" rel="noreferrer" target="_blank">Makefile.glsl.am</a> and <a href="http://Makefile.nir.am" rel="noreferrer" target="_blank">Makefile.nir.am</a>, and it's even spread to<br>
freedreno and i965.<br>
</blockquote></div><br></div><div class="gmail_extra">Sure.  I don't care alot either way.<br><br></div><div class="gmail_extra">Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><br></div><div class="gmail_extra">If someone decides that the .tmp trick is better, they can go fix up the build system and change them all including this one.<br></div></div>