Re: [cmake-developers] Remove cmProperty class

2012-02-27 Thread Brad King

On 2/25/2012 3:07 AM, Yury G. Kudryashov wrote:

The cmProperty class is used only in cmPropertyMap, and it seems easier to
implement this logic in cmPropertyMap.


Applied:

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=a592c451
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c89c879e

Thanks,
-Brad


--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Remove cmProperty class

2012-02-27 Thread Brad King

On 2/27/2012 8:46 AM, Brad King wrote:

On 2/25/2012 3:07 AM, Yury G. Kudryashov wrote:

The cmProperty class is used only in cmPropertyMap, and it seems easier to
implement this logic in cmPropertyMap.


Applied:

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=a592c451
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c89c879e


...and reverted:

On 2/27/2012 8:46 AM, CDash wrote:
 *Error*
 Source/cmMakefile.cxx 
(http://open.cdash.org/viewBuildError.php?type=0buildid=2044266)
 /home/kitware/Dashboards/My Tests/CMake-cont/Source/cmMakefile.cxx:1605: 
error: variable or field 'AddStringToProperty' declared void
 /home/kitware/Dashboards/My Tests/usr/bin/ar: 
CMakeFiles/CMakeLib.dir/cmMakefile.cxx.o: No such file or directory

It has a logical conflict with the target-include-directories topic.
Please rebase on that or wait for it to be merged to master.

Thanks,
-Brad
--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


[cmake-developers] Remove cmProperty class

2012-02-25 Thread Yury G. Kudryashov
The cmProperty class is used only in cmPropertyMap, and it seems easier to
implement this logic in cmPropertyMap.

--

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers