This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Set CMake policy CMP0051 to OLD
ClosedPublic

Authored by zturner on Feb 24 2015, 2:32 PM.

Details

Summary

When you use generator expressions in a library sources list,
and then later access the SOURCES property, the OLD behavior
(CMake 3.0 and earlier) would not include these expressions in
the SOURCES property. The NEW behavior (starting in CMake 3.1)
is that they do include the generator expressions in the SOURCES
property.

You can see an example of this warning occurring in practice on the
lldb bots currently.

http://lab.llvm.org:8011/builders/lldb-x86-windows-msvc/builds/2382/steps/cmake/logs/stdio

Diff Detail

Repository
rL LLVM

Event Timeline

zturner updated this revision to Diff 20628.Feb 24 2015, 2:32 PM
zturner retitled this revision from to [CMake] Set CMake policy CMP0051 to OLD.
zturner updated this object.
zturner edited the test plan for this revision. (Show Details)
zturner added reviewers: mcrosier, chandlerc.
zturner added a reviewer: beanz.
zturner added a subscriber: Unknown Object (MLST).
beanz accepted this revision.Feb 24 2015, 3:12 PM
beanz edited edge metadata.

LGTM.

As may have been noted by my comments on LLVMDev today. I really think we need to come up with a plan for updating CMake more frequently. Especially if we drop autoconf.

This revision is now accepted and ready to land.Feb 24 2015, 3:12 PM
This revision was automatically updated to reflect the committed changes.