<div dir="ltr">I have attached the output with a git-svn repo on Mac. Looks fine (except for the awkwardness of calling the macros SVN_REVISION and SVN_REPOSITORY; could you change the name of those in a follow-on patch?).<br><div><br></div><div>-- Sean Silva</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 26, 2014 at 5:09 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">Revised patch that I'm pretty sure will handle straight git.<br>
I'd appreciate somebody trying it with a git-svn clone, though.<br>
Thanks,<br>
--paulr<br>
<div class="HOEnZb"><div class="h5"><br>
> -----Original Message-----<br>
> From: Robinson, Paul<br>
> Sent: Wednesday, November 26, 2014 4:15 PM<br>
> To: Robinson, Paul; <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Subject: RE: [PATCH] Fix Windows build<br>
><br>
> With patch...<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: Wednesday, November 26, 2014 4:12 PM<br>
> > To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > Subject: [PATCH] Fix Windows build<br>
> ><br>
> > r222391 updated GetSVN.cmake to run the LLVM helper scripts<br>
> > GetSourceVersion<br>
> > and GetRepositoryPath.  However, being shell scripts, they work only on<br>
> > Unix-y hosts; on Windows they don't work, you end up with an<br>
> > SVNVersion.inc<br>
> > that looks like this:<br>
> ><br>
> > #define LLVM_REVISION ""<br>
> > #define LLVM_REPOSITORY ""<br>
> > #define SVN_REVISION ""<br>
> > #define SVN_REPOSITORY ""<br>
> ><br>
> > I've rewritten the scripts as best I can into CMake so they will work on<br>
> > Windows as well.  Now I get a better-looking SVNVersion.inc:<br>
> ><br>
> > #define LLVM_REVISION "222770"<br>
> > #define LLVM_REPOSITORY "<a href="http://llvm.org/svn/llvm-project/llvm/trunk" target="_blank">http://llvm.org/svn/llvm-project/llvm/trunk</a>"<br>
> > #define SVN_REVISION "222770"<br>
> > #define SVN_REPOSITORY "<a href="http://llvm.org/svn/llvm-project/cfe/trunk" target="_blank">http://llvm.org/svn/llvm-project/cfe/trunk</a>"<br>
> ><br>
> > I can only test it on SVN, but I was able to clone code from another<br>
> > CMake script so I'm pretty hopeful that it works for Git as well.<br>
> ><br>
> > Thanks,<br>
> > --paulr<br>
> ><br>
> ><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">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><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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>
<br></blockquote></div><br></div>