Going to put this in today then if there are no further issues.  This gives me time to revert today if there are still issues / disagreements, and if anything comes up next week feel free to revert then since I will be out all week.<br><br><div class="gmail_quote">On Thu Dec 18 2014 at 10:28:43 AM Sean Callanan <<a href="mailto:scallanan@apple.com">scallanan@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think it’d be nice not to delete the Makefile… it can be useful for reproducing tests if they fail, removing one reason for manually checking in Makefiles.<br>
<br>
I think, though, that CleanMakefile() should stay in but have an empty implementation, so that someone who doesn’t want the Makefiles sticking around can set an environment variable to delete them (You don’t have to actually implement that part, I can do it later, but I’d rather have CleanMakefile() around so I know where to do it.)<br>
<br>
Sean<br>
<br>
> On Dec 15, 2014, at 11:33 AM, Greg Clayton <<a href="mailto:clayborg@gmail.com" target="_blank">clayborg@gmail.com</a>> wrote:<br>
><br>
> Shouldn't the fact that the make file exists be stored in a class variable and then the class will still call CleanMakefile() if the make file didn't exist?<br>
><br>
><br>
> <a href="http://reviews.llvm.org/D6664" target="_blank">http://reviews.llvm.org/D6664</a><br>
><br>
> EMAIL PREFERENCES<br>
>  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
><br>
><br>
><br>
> ______________________________<u></u>_________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div>