<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">This was my bad. The two ifs were meant to be wrapped in if(CLANG_APPEND_VC_REV), I’ve added that in r258143.<div class=""><br class=""></div><div class="">-Chris</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jan 18, 2016, at 11:40 AM, Craig Topper <<a href="mailto:craig.topper@gmail.com" class="">craig.topper@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><span style="font-size:12.8px" class="">CLANG_APPEND_VC_REV doesn't appear to be checked anywhere. Were the two ifs supposed to check it instead of SVN_VERSION? The way it is now if the first if body executes, the second if does too since add_version_info_from_vcs sets SVN_VERSION and thus satisfies the second if.</span></div><div class="gmail_extra"><br class=""><div class="gmail_quote">On Mon, Jan 18, 2016 at 6:06 AM, Daniel Sanders via cfe-commits <span dir="ltr" class=""><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" class="">cfe-commits@lists.llvm.org</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks, that did the trick. I've removed the workaround from LLVMLinux.<br class="">
<span class="im HOEnZb"><br class="">
> -----Original Message-----<br class="">
> From: <a href="mailto:cbieneman@apple.com" class="">cbieneman@apple.com</a> [mailto:<a href="mailto:cbieneman@apple.com" class="">cbieneman@apple.com</a>] On Behalf Of<br class="">
> Chris Bieneman<br class="">
</span><span class="im HOEnZb">> Sent: 15 January 2016 17:55<br class="">
> To: Daniel Sanders<br class="">
> Cc: <a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class="">
> Subject: Re: r257827 - [CMake] Set SVN_REVISION if<br class="">
> CLANG_APPEND_VC_REV=On<br class="">
><br class="">
</span><div class="HOEnZb"><div class="h5">> Thanks for the heads up. It looks like that module is was excluded from the<br class="">
> LLVM install. I’ve changed that in LLVM r257909. That change should resolve<br class="">
> your build issue. Please let me know if you continue having problems.<br class="">
><br class="">
> Thanks,<br class="">
> -Chris<br class="">
><br class="">
> > On Jan 15, 2016, at 5:18 AM, Daniel Sanders <<a href="mailto:Daniel.Sanders@imgtec.com" class="">Daniel.Sanders@imgtec.com</a>><br class="">
> wrote:<br class="">
> ><br class="">
> > Hi Chris,<br class="">
> ><br class="">
> > This doesn't seem to work when building clang separately from llvm.<br class="">
> LLVMLinux fails to build clang with:<br class="">
> > CMake Error at CMakeLists.txt:104 (include):<br class="">
> > include could not find load file:<br class="">
> ><br class="">
> > VersionFromVCS<br class="">
> ><br class="">
> > CMake Error at CMakeLists.txt:222 (add_version_info_from_vcs):<br class="">
> > Unknown CMake command "add_version_info_from_vcs".<br class="">
> > See<br class="">
> <a href="http://buildbot.llvm.linuxfoundation.org/builders/13_malta/builds/383/step" rel="noreferrer" target="_blank" class="">http://buildbot.llvm.linuxfoundation.org/builders/13_malta/builds/383/step</a><br class="">
> s/shell_3/logs/stdio for the full log.<br class="">
> ><br class="">
> > I've added a patch to llvmlinux to work around the problem for now<br class="">
> <a href="http://git.linuxfoundation.org/?p=llvmlinux.git;a=blob;f=toolchain/clang/pat" rel="noreferrer" target="_blank" class="">http://git.linuxfoundation.org/?p=llvmlinux.git;a=blob;f=toolchain/clang/pat</a><br class="">
> ches/clang/workaround-<br class="">
> versionfromvcsbug.patch;h=848a096df37b1255575650680a266234f5d4936e;h<br class="">
> b=e0c4c72c5a008006dc230db748ea69e0d1518daf.<br class="">
> > Should we make that change to clang or fix it another way?<br class="">
> ><br class="">
> >> -----Original Message-----<br class="">
> >> From: cfe-commits [mailto:<a href="mailto:cfe-commits-bounces@lists.llvm.org" class="">cfe-commits-bounces@lists.llvm.org</a>] On<br class="">
> Behalf<br class="">
> >> Of Chris Bieneman via cfe-commits<br class="">
> >> Sent: 14 January 2016 22:45<br class="">
> >> To: <a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class="">
> >> Subject: r257827 - [CMake] Set SVN_REVISION if<br class="">
> >> CLANG_APPEND_VC_REV=On<br class="">
> >><br class="">
> >> Author: cbieneman<br class="">
> >> Date: Thu Jan 14 16:45:12 2016<br class="">
> >> New Revision: 257827<br class="">
> >><br class="">
> >> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=257827&view=rev" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project?rev=257827&view=rev</a><br class="">
> >> Log:<br class="">
> >> [CMake] Set SVN_REVISION if CLANG_APPEND_VC_REV=On<br class="">
> >><br class="">
> >> This matches autoconf's ability to put clang revisions in the clang --version<br class="">
> >> spew.<br class="">
> >><br class="">
> >> Modified:<br class="">
> >> cfe/trunk/CMakeLists.txt<br class="">
> >><br class="">
> >> Modified: cfe/trunk/CMakeLists.txt<br class="">
> >> URL: <a href="http://llvm.org/viewvc/llvm-" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-</a><br class="">
> >><br class="">
> project/cfe/trunk/CMakeLists.txt?rev=257827&r1=257826&r2=257827&view<br class="">
> >> =diff<br class="">
> >><br class="">
> ==========================================================<br class="">
> >> ====================<br class="">
> >> --- cfe/trunk/CMakeLists.txt (original)<br class="">
> >> +++ cfe/trunk/CMakeLists.txt Thu Jan 14 16:45:12 2016<br class="">
> >> @@ -101,6 +101,7 @@ if( CMAKE_SOURCE_DIR STREQUAL CMAKE_CURR<br class="">
> >> include(AddLLVM)<br class="">
> >> include(TableGen)<br class="">
> >> include(HandleLLVMOptions)<br class="">
> >> + include(VersionFromVCS)<br class="">
> >><br class="">
> >> set(PACKAGE_VERSION "${LLVM_PACKAGE_VERSION}")<br class="">
> >><br class="">
> >> @@ -213,6 +214,18 @@ if(CLANG_REPOSITORY_STRING)<br class="">
> >> add_definitions(-<br class="">
> >> DCLANG_REPOSITORY_STRING="${CLANG_REPOSITORY_STRING}")<br class="">
> >> endif()<br class="">
> >><br class="">
> >> +option(CLANG_APPEND_VC_REV<br class="">
> >> + "Append the version control system revision id to clang version spew"<br class="">
> >> OFF)<br class="">
> >> +<br class="">
> >> +if(NOT SVN_REVISION)<br class="">
> >> + # This macro will set SVN_REVISION in the parent scope<br class="">
> >> + add_version_info_from_vcs(VERSION_VAR)<br class="">
> >> +endif()<br class="">
> >> +<br class="">
> >> +if(SVN_REVISION)<br class="">
> >> + add_definitions(-DSVN_REVISION="${SVN_REVISION}")<br class="">
> >> +endif()<br class="">
> >> +<br class="">
> >> set(CLANG_VENDOR_UTI "org.llvm.clang" CACHE STRING<br class="">
> >> "Vendor-specific uti.")<br class="">
> >><br class="">
> >><br class="">
> >><br class="">
> >> _______________________________________________<br class="">
> >> cfe-commits mailing list<br class="">
> >> <a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class="">
> >> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="">
<br class="">
_______________________________________________<br class="">
cfe-commits mailing list<br class="">
<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class="">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="">
</div></div></blockquote></div><br class=""><br clear="all" class=""><div class=""><br class=""></div>-- <br class=""><div class="gmail_signature">~Craig</div>
</div>
</div></blockquote></div><br class=""></div></body></html>