[llvm-commits] [llvm] r108416 - /llvm/trunk/Makefile.rules

[Eric Christopher]

On Jul 15, 2010, at 1:21 AM, Eric Christopher wrote:

> 
> On Jul 14, 2010, at 11:36 PM, Ted Kremenek wrote:
> 
>> Author: kremenek
>> Date: Thu Jul 15 01:36:57 2010
>> New Revision: 108416
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=108416&view=rev
>> Log:
>> Correctly set rpath on Mac OS X for executable tools.  Note that I am not certain this is the best fix; the code immediately above looks confused, as it first checks to see if we are NOT on Darwin and then checks the Darwin version number.  This fix allows c-index-test (in Clang) to run outside of running the regression test suite.  I would appreciate if someone reviewed this.
> 
> Yeah, line 625 can probably go away.  I'll test that in the morning.

Seems fine.  I've removed the conditional and committed.  Thanks Ted!

-eric