[cmake-developers] [PATCH 2/2] Remove cmProperty.{h,cxx}

Yury G. Kudryashov urkud.urkud at gmail.com
Tue Feb 28 12:15:52 EST 2012


David Cole wrote:

> What's the motivation here?
The motivation was to remove a class that has no useful methods. The only 
useful method was AppendProperty, and it can be easily implemented in 
cmPropertyMap.

> 
> This will likely conflict (logically, at compile time) with the
> recently merged-to-next topic for the target-include-directories...
I'd move AddStringToProperty to cmPropertyMap (say, 
cmPropertyMap::PrependProperty). I'll prepare a patch in a few days (rather 
busy now).
-- 
Yury G. Kudryashov,
mailto: urkud at mccme.ru




More information about the cmake-developers mailing list