[cmake-developers] Adding push/pop_required_vars() to cmake

[Alexander Neundorf]

Hi,

in KDE we have two macros macro_push_required_vars() and 
macro_pop_required_vars().
They are intended to be used before setting one or more of the 
CMAKE_REQUIRED_INCLUDES/DEFINITIONS/etc. variables for a specific test, so 
that after the check the previous state can be easily restored. It works 
multilevel:
https://projects.kde.org/projects/kde/kdelibs/repository/revisions/master/entry/cmake/modules/MacroPushRequiredVars.cmake

Last time Brad suggested to rename them to push/pop_check_state():
http://www.cmake.org/pipermail/cmake-developers/2006-September/000137.html

I don't have a strong opinion on the naming.
push/pop_required_vars() or push/pop_check_state() ? Adding a cmake_ prefix ?
And put them into CMakePushCheckState.cmake ?

Alex
___
cmake-developers mailing list
cmake-developers@cmake.org
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Re: [cmake-developers] Adding push/pop_required_vars() to cmake

[Brad King]

On 7/6/2011 3:53 PM, Alexander Neundorf wrote:
 I don't have a strong opinion on the naming.

Neither do I.  I'd just like a cmake_ prefix.

 And put them into CMakePushCheckState.cmake ?

Sure.

-Brad
___
cmake-developers mailing list
cmake-developers@cmake.org
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers