Code review comment for lp:~widelands-dev/widelands/cmakepolicy

Toni Förster (stonerl) wrote :

@kaputtnik the changes I made set the policy to NEW. The trick that it. did were the additional linker options. But it broke travis builds and will throw errors on system that use old cmake version.

Your proposed switch sets it to the old (deprecated) behavior. Would you be so nice and test the latest change I pushed?

This change only sets the policy to NEW, if it is known to the Cmake version in use. This should work with travis, but we have to wait, though.

« Back to merge proposal