[Cmake-commits] [cmake-commits] king committed cmGetPropertyCommand.cxx 1.10 1.11

cmake-commits at cmake.org cmake-commits at cmake.org
Mon Mar 9 17:57:14 EDT 2009


Update of /cvsroot/CMake/CMake/Source
In directory public:/mounts/ram/cvs-serv18730/Source

Modified Files:
	cmGetPropertyCommand.cxx 
Log Message:
BUG: Fix get_property result for bad property

When a property does not exist we are supposed to return an empty value.
Previously if a property did not exist we just left the value of the
output variable unchanged.  This teaches CMake to remove the definition
of the output variable in this case.


Index: cmGetPropertyCommand.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/cmGetPropertyCommand.cxx,v
retrieving revision 1.10
retrieving revision 1.11
diff -C 2 -d -r1.10 -r1.11
*** cmGetPropertyCommand.cxx	5 Jan 2009 20:00:56 -0000	1.10
--- cmGetPropertyCommand.cxx	9 Mar 2009 21:57:12 -0000	1.11
***************
*** 206,210 ****
    else // if(this->InfoType == OutValue)
      {
!     this->Makefile->AddDefinition(this->Variable.c_str(), value);
      }
    return true;
--- 206,217 ----
    else // if(this->InfoType == OutValue)
      {
!     if(value)
!       {
!       this->Makefile->AddDefinition(this->Variable.c_str(), value);
!       }
!     else
!       {
!       this->Makefile->RemoveDefinition(this->Variable.c_str());
!       }
      }
    return true;



More information about the Cmake-commits mailing list