[PATCH] Fix test-release.sh to use DESTDIR

Hans Wennborg hans at chromium.org
Tue Jul 14 13:03:54 PDT 2015


On Tue, Jul 14, 2015 at 12:50 PM, Dan Liew <dan at su-root.co.uk> wrote:
>> I did a 64-bit build on my Linux box and everything built and passed
>> tests correctly.
>>
>> LLVM_INSTALL_PREFIX is set to /usr/local in LLVMConfig.cmake in the
>> tarball, so that seems right too.
>>
>> Please go ahead and commit.
>
> Great. Thanks for the review.
>
> Committed in r242187
>
> On a related note is this the version that will be used for the 3.7
> release? I seem remember there being a patch floating around in
> phabricator for this script to use CMake to do the build instead.

Yes. 3.7 hasn't been branched yet (will happen later today,
hopefully), so this is the version that will get used. I'm also hoping
to get the patch for using CMake (http://reviews.llvm.org/D10715)
landed soon.

Thanks,
Hans



More information about the llvm-commits mailing list