<div dir="ltr">Resending: just noticed that Kevin wasn't on the CC list of my last message.<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 19, 2013 at 10:31 PM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5">On Tue, Mar 19, 2013 at 5:05 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br>
<div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br>
On Mar 19, 2013, at 11:08 AM, Stefanus Du Toit <<a href="mailto:stefanus.du.toit@intel.com" target="_blank">stefanus.du.toit@intel.com</a>> wrote:<br>
<br>
><br>
>  If the script works today, rewriting it in Python now seems unnecessary. Sounds like Kevin will use Python for future scripts, but since this is just a little utility, I think it's fine.<br>
<br>
</div>+1<br>
<br>
That shell script looks much nicer than most of our Python scripts.<br>
<span><font color="#888888"><br>
/jakob<br>
</font></span><div><div><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div></div><div class="gmail_extra">Alright it looks like there is consensus to put the script in as-is.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Kevin, please update the patch to address my comment about x-ref'ing the script from the file header, and the typo I just caught (on phab). Do you need me to commit this for you?</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Now that I think about it, AFAIK currently the only LLVM development environment where a POSIX-compatible shell would not be available is for people developing in Visual Studio, and I don't think they are very interested in ctags :P </div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Also, Kevin, would you care to enlighten us mere mortals where you learned the black art of writing readable shell scripts?</div><span class="HOEnZb"><font color="#888888"><div class="gmail_extra">

<br></div><div class="gmail_extra">-- Sean Silva</div></font></span></div>
</blockquote></div><br></div></div>