[LNT] Parse --version correctly from CMake builds

Tilmann Scheller tilmann.scheller at gmail.com
Tue Jun 17 11:02:39 PDT 2014


Hi,

actually there's a patch pending for review which improves the version info
for CMake builds:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140526/219022.html

Maybe this patch already solves the problem?

Regards,

Tilmann

On Tuesday, June 17, 2014, Duncan P. N. Exon Smith <dexonsmith at apple.com>
wrote:

> On 2014-Jun-17, at 05:21, James Molloy <james.molloy at arm.com
> <javascript:;>> wrote:
> >
> > Hi Yi, Chris,
> >
> > LNT currently fails to parse clang’s –version string correctly for CMake
> builds. CMake builds do not emit the branch name, they only emit the
> revision. Currently LNT thinks the revision number is the branch name,
> which also means it fails to infer the run_order correctly for submission.
> >
> > The attached simple patch fixes this.
> >
> > OK to commit?
> >
>
> I don't know much about LNT, but this seems like the wrong fix to me.
> Shouldn't we be fixing the build systems to match?
>
> In particular, isn't this a bug in CMake builds?
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu <javascript:;>
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140617/6da996df/attachment.html>


More information about the llvm-commits mailing list