Yea, and that's how the patch is implemented now.  But I think what Greg is saying is that he thought it should check for an existing Makefile first.  If it exists, make a copy of it, then generate the makefile, then after everything is done restore the old Makefile.  Do you think that's important?  You wrote most of the lldbinline stuff it looks like, so ultimately if you're ok with it then I am too, but just want to make sure we're all on the same page.<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>