<div dir="ltr">Hi Paul,<div><br></div><div>That looks fine to me, thanks.</div><div><br></div><div>FWIW, I'm not really much of a fan of the current implementation (prior to your path), I would like it better if lit controlled more tightly exactly what executables were found and had a list of allowed directories (with exceptions, maybe) instead of a list of explicit tool names.</div>
<div><br></div><div> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 19, 2014 at 3:34 PM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping*3.<br>
<div class="im HOEnZb"><br>
> -----Original Message-----<br>
> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-">llvm-commits-</a><br>
> <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Robinson, Paul<br>
</div><div class="HOEnZb"><div class="h5">> Sent: Wednesday, March 12, 2014 2:37 PM<br>
> To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a> commits (<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>);<br>
> <a href="mailto:daniel@zuster.org">daniel@zuster.org</a><br>
> Subject: RE: [PATCH] Refactor lit.cfg to use lit.util.which<br>
><br>
> Ping*2. +Daniel Dunbar as LIT owner, hope that's okay.<br>
><br>
> I want to use this as the basis for adding equivalent code<br>
> to Clang's lit.cfg which currently doesn't do the tool name<br>
> substitution thing.  We really need that for our builds.<br>
> Thanks,<br>
> --paulr<br>
><br>
> > -----Original Message-----<br>
> > From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-">llvm-commits-</a><br>
> > <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Robinson, Paul<br>
> > Sent: Friday, March 07, 2014 9:45 AM<br>
> > To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a> commits (<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>)<br>
> > Subject: RE: [PATCH] Refactor lit.cfg to use lit.util.which<br>
> ><br>
> > Ping. LIT owner? not sure who.<br>
> > --paulr<br>
> ><br>
> > > -----Original Message-----<br>
> > > From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-">llvm-commits-</a><br>
> > > <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Robinson, Paul<br>
> > > Sent: Tuesday, March 04, 2014 5:01 PM<br>
> > > To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a> commits (<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>)<br>
> > > Subject: [PATCH] Refactor lit.cfg to use lit.util.which<br>
> > ><br>
> > > llvm/test/lit.cfg was doing tool-name substitutions using PATHEXT<br>
> > > and os.path.exists().  Better to use lit.util.which, as a<br>
> centralized<br>
> > > place that knows path-name quirks.<br>
> > > As a side effect, it's easy to tell if a tool doesn't exist, so I<br>
> > > removed some non-existent tools from the list.  (Possibly 'clang'<br>
> > > should also be removed? It shouldn't be used for any LLVM tests...)<br>
> > ><br>
> > > Okay to submit?<br>
> > > Thanks,<br>
> > > --paulr<br>
<br>
</div></div></blockquote></div><br></div>