diff options
author | Andy Cedilnik <andy.cedilnik@kitware.com> | 2004-04-15 17:58:10 (GMT) |
---|---|---|
committer | Andy Cedilnik <andy.cedilnik@kitware.com> | 2004-04-15 17:58:10 (GMT) |
commit | 504d0bc3b38574d8394995b589214ce9cb0f6d3a (patch) | |
tree | ab49a624e100b25ca9ea5bb48624cc8d2e110857 | |
parent | a5c3087360b79132c7e3c5004e6a9d14c7e62805 (diff) | |
download | CMake-504d0bc3b38574d8394995b589214ce9cb0f6d3a.zip CMake-504d0bc3b38574d8394995b589214ce9cb0f6d3a.tar.gz CMake-504d0bc3b38574d8394995b589214ce9cb0f6d3a.tar.bz2 |
ENH: ADD REMOVE_DEFINITION command. Fix feature request: Bug #182 - Add opposite to ADD_DEFINITIONS
-rw-r--r-- | Source/cmCommands.cxx | 2 | ||||
-rw-r--r-- | Source/cmMakefile.cxx | 5 | ||||
-rw-r--r-- | Source/cmMakefile.h | 1 | ||||
-rw-r--r-- | Source/cmRemoveDefinitionsCommand.cxx | 35 | ||||
-rw-r--r-- | Source/cmRemoveDefinitionsCommand.h | 82 |
5 files changed, 125 insertions, 0 deletions
diff --git a/Source/cmCommands.cxx b/Source/cmCommands.cxx index 9671bb6..4a7417a 100644 --- a/Source/cmCommands.cxx +++ b/Source/cmCommands.cxx @@ -58,6 +58,7 @@ #include "cmMessageCommand.cxx" #include "cmOptionCommand.cxx" #include "cmProjectCommand.cxx" +#include "cmRemoveDefinitionsCommand.cxx" #include "cmSeparateArgumentsCommand.cxx" #include "cmSetCommand.cxx" #include "cmSetSourceFilesPropertiesCommand.cxx" @@ -150,6 +151,7 @@ void GetPredefinedCommands(std::list<cmCommand*>& commands) commands.push_back(new cmMessageCommand); commands.push_back(new cmOptionCommand); commands.push_back(new cmProjectCommand); + commands.push_back(new cmRemoveDefinitionsCommand); commands.push_back(new cmSeparateArgumentsCommand); commands.push_back(new cmSetCommand); commands.push_back(new cmSetSourceFilesPropertiesCommand); diff --git a/Source/cmMakefile.cxx b/Source/cmMakefile.cxx index f0e187f..4f287f6 100644 --- a/Source/cmMakefile.cxx +++ b/Source/cmMakefile.cxx @@ -766,6 +766,11 @@ void cmMakefile::AddDefineFlag(const char* flag) } +void cmMakefile::RemoveDefineFlag(const char* flag) +{ + cmSystemTools::ReplaceString(m_DefineFlags, flag, " "); +} + void cmMakefile::AddLinkLibrary(const char* lib, cmTarget::LinkLibraryType llt) { m_LinkLibraries.push_back( diff --git a/Source/cmMakefile.h b/Source/cmMakefile.h index dbfe6a6..092d742 100644 --- a/Source/cmMakefile.h +++ b/Source/cmMakefile.h @@ -163,6 +163,7 @@ public: * Add a define flag to the build. */ void AddDefineFlag(const char* definition); + void RemoveDefineFlag(const char* definition); /** * Add an executable to the build. diff --git a/Source/cmRemoveDefinitionsCommand.cxx b/Source/cmRemoveDefinitionsCommand.cxx new file mode 100644 index 0000000..1532742 --- /dev/null +++ b/Source/cmRemoveDefinitionsCommand.cxx @@ -0,0 +1,35 @@ +/*========================================================================= + + Program: CMake - Cross-Platform Makefile Generator + Module: $RCSfile$ + Language: C++ + Date: $Date$ + Version: $Revision$ + + Copyright (c) 2002 Kitware, Inc., Insight Consortium. All rights reserved. + See Copyright.txt or http://www.cmake.org/HTML/Copyright.html for details. + + This software is distributed WITHOUT ANY WARRANTY; without even + the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR + PURPOSE. See the above copyright notices for more information. + +=========================================================================*/ +#include "cmRemoveDefinitionsCommand.h" + +// cmRemoveDefinitionsCommand +bool cmRemoveDefinitionsCommand::InitialPass(std::vector<std::string> const& args) +{ + // it is OK to have no arguments + if(args.size() < 1 ) + { + return true; + } + + for(std::vector<std::string>::const_iterator i = args.begin(); + i != args.end(); ++i) + { + m_Makefile->RemoveDefineFlag(i->c_str()); + } + return true; +} + diff --git a/Source/cmRemoveDefinitionsCommand.h b/Source/cmRemoveDefinitionsCommand.h new file mode 100644 index 0000000..ac1c75c --- /dev/null +++ b/Source/cmRemoveDefinitionsCommand.h @@ -0,0 +1,82 @@ +/*========================================================================= + + Program: CMake - Cross-Platform Makefile Generator + Module: $RCSfile$ + Language: C++ + Date: $Date$ + Version: $Revision$ + + Copyright (c) 2002 Kitware, Inc., Insight Consortium. All rights reserved. + See Copyright.txt or http://www.cmake.org/HTML/Copyright.html for details. + + This software is distributed WITHOUT ANY WARRANTY; without even + the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR + PURPOSE. See the above copyright notices for more information. + +=========================================================================*/ +#ifndef cmRemoveDefinitionsCommand_h +#define cmRemoveDefinitionsCommand_h + +#include "cmCommand.h" + +/** \class cmRemoveDefinitionsCommand + * \brief Specify a list of compiler defines + * + * cmRemoveDefinitionsCommand specifies a list of compiler defines. These defines will + * be removed from the compile command. + */ +class cmRemoveDefinitionsCommand : public cmCommand +{ +public: + /** + * This is a virtual constructor for the command. + */ + virtual cmCommand* Clone() + { + return new cmRemoveDefinitionsCommand; + } + + /** + * This is called when the command is first encountered in + * the CMakeLists.txt file. + */ + virtual bool InitialPass(std::vector<std::string> const& args); + + /** + * This determines if the command gets propagated down + * to makefiles located in subdirectories. + */ + virtual bool IsInherited() {return true;} + + /** + * The name of the command as specified in CMakeList.txt. + */ + virtual const char* GetName() {return "REMOVE_DEFINITIONS";} + + /** + * Succinct documentation. + */ + virtual const char* GetTerseDocumentation() + { + return "Removes -D define flags to the command line of C and C++ compilers."; + } + + /** + * More documentation. + */ + virtual const char* GetFullDocumentation() + { + return + " REMOVE_DEFINITIONS(-DFOO -DBAR ...)\n" + "Removes flags from command line of C and C++ compilers. " + "This command can be used to remove any flag from a compile line, " + "but the -D flag is accepted most C/C++ compilers. " + "Other flags may not be as portable."; + } + + cmTypeMacro(cmRemoveDefinitionsCommand, cmCommand); +}; + + + +#endif |