[CMake] CTestDashboardTargets targets should be captured under PREDEFINED_TARGETS_FOLDER?

2011-02-20 Thread Johan Björk
Hey guys,

I am not quite sure if PREDEFINED_TARGETS_FOLDER is supposed to also contain
targets created by external modules (CTest). What do you guys think? Right
now there seems to be no straightforward way to move the
CTestDashboardTargets folder away from the other toplevel targets.
Should be trivial to have it honor PREDEFINED_TARGETS_FOLDER, if that's the
desired behavior.

Cheers
/Johan
___
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://www.cmake.org/mailman/listinfo/cmake

Re: [CMake] CTestDashboardTargets targets should be captured under PREDEFINED_TARGETS_FOLDER?

2011-02-20 Thread David Cole
In CTestTargets.cmake (included from CTest.cmake), there are lines that read:

  SET_PROPERTY(TARGET ... PROPERTY FOLDER CTestDashboardTargets)

That's the thing that assigns that folder value to those targets. It's
explicitly separated from the CMake predefined targets folder because
they are two separate groups of things.

You can change them on a per-project basis if you want by re-assigning
the target property for those targets to whatever value you want after
including CTest.cmake.

On the other hand, if everybody here has the same opinions as Johan, I
have no problem changing the default value for this folder to be under
the predefined folders target.


HTH,
David


On Sun, Feb 20, 2011 at 8:56 AM, Johan Björk p...@spotify.com wrote:
 Hey guys,
 I am not quite sure if PREDEFINED_TARGETS_FOLDER is supposed to also contain
 targets created by external modules (CTest). What do you guys think? Right
 now there seems to be no straightforward way to move the
 CTestDashboardTargets folder away from the other toplevel targets.
 Should be trivial to have it honor PREDEFINED_TARGETS_FOLDER, if that's the
 desired behavior.
 Cheers
 /Johan

 ___
 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://www.cmake.org/mailman/listinfo/cmake

___
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://www.cmake.org/mailman/listinfo/cmake