Brad King wrote:

>>>>// For backwards compatibility replace varibles in source names.
>>>>-  // This should eventually be removed.
>>>>+  // TODO: This should eventually be removed.
>>>>this->Makefile->ExpandVariablesInString(src);
>>>
>>> A policy should be added to remove this behavior.
>>
>> Any idea how to unit test this?
> 
> The above expansion makes this code work:
> 
> set(sources foo.c)
> add_executable(foo \${sources})
> 
> but shouldn't with the policy.

Confirmed. I added CMP0049 to control this behavior.

Thanks,

Steve.



-- 

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to