[cmake-developers] A variable for the role of the man who runs cmake?

2013-08-20 Thread Yury G. Kudryashov
;Packager;Maintainer;User) but in this case a common name will help users (at least, once buildsystems will start really use this variable). -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource

[cmake-developers] [PATCH] Fix a typo in documentation

2013-08-15 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com --- Source/CPack/cmCPackDocumentVariables.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/CPack/cmCPackDocumentVariables.cxx b/Source/CPack/cmCPackDocumentVariables.cxx index 1dfaaf9..8b16ae9 100644 --- a/Source/CPack

[cmake-developers] [PATCH] spell: fix a few typos in comments

2013-01-21 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com --- Source/cmMakefile.cxx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Source/cmMakefile.cxx b/Source/cmMakefile.cxx index b432986..d1dc5a2 100644 --- a/Source/cmMakefile.cxx +++ b/Source/cmMakefile.cxx @@ -2243,7

Re: [cmake-developers] add_library(OBJECT) and AUTOMOC

2013-01-14 Thread Yury G. Kudryashov
2013/1/14 Alexander Neundorf neund...@kde.org: On Friday 11 January 2013, Brad King wrote: On 01/11/2013 04:47 AM, Yury G. Kudryashov wrote: Look at $ git pull gitorious:~urkud1/cmake/urkud-cmake.git w/automoc-object please. The commit message for the first commit seems incomplete. Also

[cmake-developers] add_library(OBJECT) and AUTOMOC

2013-01-11 Thread Yury G. Kudryashov
) The second commit in the branch updates Tests/QtAutomoc. -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- 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

Re: [cmake-developers] Exporting dependent library targets in multiple export sets

2012-09-17 Thread Yury G. Kudryashov
2012/9/17 Brad King brad.k...@kitware.com: On 09/15/2012 04:45 PM, Alexander Neundorf wrote: Also, one of Yury's commit messages says: TODO: remove another clear() because it doesn't delete TargetExports. What is that about? clear() on a vector of TargetExport* does not call delete on each

Re: [cmake-developers] Exporting dependent library targets in multiple export sets

2012-09-11 Thread Yury G. Kudryashov
2012/9/11 Alexander Neundorf neund...@kde.org: Hi Yury, will you be able to work on this for the 2.8.10 release ? What are the deadlines? -- Yours Yury, mailto: urkud.ur...@gmail.com -- Powered by www.kitware.com Visit other Kitware open-source projects at

Re: [cmake-developers] Exporting dependent library targets in multiple export sets

2012-08-23 Thread Yury G. Kudryashov
Hi! 2012/8/23 Daniel Pfeifer dan...@pfeifer-mail.de: 2012/8/22 Yury G. Kudryashov urkud.ur...@gmail.com: Any latest install strategy is bound to generate subtle problems. Dependencies among export sets should be explicit: install(EXPORT MyExport DEPENDS YourExport) How should I choose

Re: [cmake-developers] Exporting dependent library targets in multiple export sets

2012-08-22 Thread Yury G. Kudryashov
Hi, 2012/8/23 Alexander Neundorf neund...@kde.org: Hi, If I would do this, I would get the error for the BarExport export set, that it does not contain target foo, which it depends on. We'd like to make this work, since we need this in KDE. How can this be done ? Sorry for the very long

Re: [cmake-developers] Exporting dependent library targets in multiple export sets

2012-08-22 Thread Yury G. Kudryashov
2012/8/23 Brad King brad.k...@kitware.com: On 08/22/2012 04:57 PM, Yury G. Kudryashov wrote: I'm going to push the current state of my work to git pull git://gitorious.org/~urkud1/cmake/urkud-cmake.git w/export-set in a few minutes. Thanks for working on this! It should work, though I

Re: [cmake-developers] Welcome to June, time for the next release candidate

2012-06-05 Thread Yury G. Kudryashov
, but new ones are needed. Hopefully we can get it ready early in the next cycle. The problem is that I have a few deadlines in June: 2 grant applications have deadlines on 15th, and one on 20th... I'll try to find some time for cmake, but I'm not sure... -- Yury G. Kudryashov, mailto: ur

Re: [cmake-developers] FindCXX11.cmake

2012-04-09 Thread Yury G. Kudryashov
AND required features check_cxx11_features( REQUIRED required1 required2 OPTIONAL optional1 optional2) -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please

[cmake-developers] FindCXX11.cmake

2012-04-04 Thread Yury G. Kudryashov
: - for each of supported compilers other than gcc and clang, which flags should I check? - for each new language feature, do you have a minimal code example that check if this feature is available? Current version of FindCXX11.cmake is attached. This version is not ready for review. -- Yury G

Re: [cmake-developers] The lib64 case

2012-03-26 Thread Yury G. Kudryashov
: /opt/p1, /opt/p1/foo, /usr/include, /usr/include/foo, /opt/p2, /opt/p2/foo. -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- 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

Re: [cmake-developers] Doxygen documentation

2012-03-06 Thread Yury G. Kudryashov
Brad King wrote: On 3/6/2012 2:23 AM, Yury G. Kudryashov wrote: Hi! I've just found that there are two doxygen configs in cmake source tree: * cmake/doxygen.config I think you can remove this one. * cmake/Utilities/Doxygen/* VTK magic The second one can be enabled

[cmake-developers] Doxygen documentation

2012-03-05 Thread Yury G. Kudryashov
Hi! I've just found that there are two doxygen configs in cmake source tree: * cmake/doxygen.config * cmake/Utilities/Doxygen/* VTK magic The second one can be enabled by BUILD_DOCUMENTATION but doc_makeall.sh references some VTK_* variables. Which one should I use/fix/..? -- Yury G

Re: [cmake-developers] Introducing: message(DEBUG)

2012-03-03 Thread Yury G. Kudryashov
in mysubdir/CMakeLists.txt -DCMAKE_DEBUG_MESSAGES_mysubdir=ON # Turn all debug messages in **/CMakeLists.txt -DCMAKE_DEBUG_MESSAGES_CMakeLists=ON -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com

[cmake-developers] Two more patches

2012-03-03 Thread Yury G. Kudryashov
://gitorious.org/~urkud1/cmake/urkud-cmake/commits/rename-used-commands -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- 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

Re: [cmake-developers] Two pull requests

2012-02-29 Thread Yury G. Kudryashov
Brad King wrote: On 2/28/2012 4:21 PM, Brad King wrote: On 2/28/2012 4:02 PM, Yury G. Kudryashov wrote: git pull git://gitorious.org/~urkud1/cmake/urkud-cmake.git doxygen-fixes Our style checker limits .h and .cxx files to 79 columns. Some of the updated comments exceed this limit. Please

Re: [cmake-developers] Two pull requests

2012-02-29 Thread Yury G. Kudryashov
Brad King wrote: On 2/28/2012 4:21 PM, Brad King wrote: On 2/28/2012 4:02 PM, Yury G. Kudryashov wrote: git pull git://gitorious.org/~urkud1/cmake/urkud-cmake.git run-vim-spellcheck Merged, thanks: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=adc2c990 git pull git

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

2012-02-28 Thread Yury G. Kudryashov
-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: ur...@mccme.ru -- Powered by www.kitware.com Visit other Kitware open-source

[cmake-developers] Two pull requests

2012-02-28 Thread Yury G. Kudryashov
, or should I send e-mails like this? I don't want to fill a bug report in mantis for each small fix. -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic

Re: [cmake-developers] Two pull requests

2012-02-28 Thread Yury G. Kudryashov
Brad King wrote: On 2/28/2012 2:46 PM, Yury G. Kudryashov wrote: git pull git://gitorious.org/~urkud1/cmake/urkud-cmake.git ready/fix-typos Please combine those commits and write a single commit message that briefly explains the tools you ran to find the errors. I've also renamed branch

Re: [cmake-developers] [PATCH 1/2] cmPropertyDefinition::IsChained is const

2012-02-28 Thread Yury G. Kudryashov
Brad King wrote: On 2/25/2012 3:19 AM, Yury G. Kudryashov wrote: Yury G. Kudryashov wrote: From: Yury G. Kudryashov urkud.ur...@gmail.com Mailman says that the next patch is too big. The compressed version is attached. While building I get: This branch compiles with -Werror here: git

[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

[cmake-developers] [PATCH 1/2] Drop cmProperty class

2012-02-25 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com This class contains nothing but name (already stored in cmPropertyMap key) and value (string). --- Source/cmProperty.cxx | 40 Source/cmProperty.h | 24 ++-- Source

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

2012-02-25 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com After previous commit cmProperty.h contained only one enum. Move it to cmPropertyMap.h. --- Source/CMakeLists.txt |2 -- Source/cmDocumentation.h |2 +- Source/cmProperty.h | 21 - Source

[cmake-developers] [PATCH 1/2] cmPropertyDefinition::IsChained is const

2012-02-25 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com --- Source/cmPropertyDefinition.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Source/cmPropertyDefinition.h b/Source/cmPropertyDefinition.h index f68db87..58d1472 100644 --- a/Source/cmPropertyDefinition.h +++ b/Source

Re: [cmake-developers] [PATCH 1/2] cmPropertyDefinition::IsChained is const

2012-02-25 Thread Yury G. Kudryashov
Yury G. Kudryashov wrote: From: Yury G. Kudryashov urkud.ur...@gmail.com Mailman says that the next patch is too big. The compressed version is attached. -- Yury G. Kudryashov, mailto: ur...@mccme.ru 0002-Add-const-qualifier-to-some-cmCommand-members.patch.xz Description: application/xz

[cmake-developers] [PATCH] Drop if(...) check because condition is always true

2012-02-25 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com GetLocation returns std::string::c_str() which is never NULL --- Source/cmLocalGenerator.cxx |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/Source/cmLocalGenerator.cxx b/Source/cmLocalGenerator.cxx index ffbeb48

[cmake-developers] Coding style questions

2012-02-24 Thread Yury G. Kudryashov
std::string and cmStdString? -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- 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

[cmake-developers] [PATCH] Fix typo in documentation

2012-02-24 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com --- Source/cmDocumentVariables.cxx |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Source/cmDocumentVariables.cxx b/Source/cmDocumentVariables.cxx index 1cab2b5..366ebaa 100644 --- a/Source/cmDocumentVariables.cxx +++ b

[cmake-developers] [PATCH] CMakeFindPackageMode.cmake: fix 32/64bit detection if `file' is a symlink

2012-02-23 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com --- Modules/CMakeFindPackageMode.cmake |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/Modules/CMakeFindPackageMode.cmake b/Modules/CMakeFindPackageMode.cmake index 4296577..bcf3014 100644 --- a/Modules

Re: [cmake-developers] Making Config.cmake files easier to write

2012-02-17 Thread Yury G. Kudryashov
with simple version logic). -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- 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

Re: [cmake-developers] Making Config.cmake files easier to write

2012-02-14 Thread Yury G. Kudryashov
to specify all required path variables explicitly. -- Yury G. Kudryashov, mailto: ur...@mccme.ru -- 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

Re: [cmake-developers] Making Config.cmake files easier to write

2012-02-14 Thread Yury G. Kudryashov
Alexander Neundorf wrote: On Tuesday 14 February 2012, Yury G. Kudryashov wrote: will substitute @PACKAGE_INCLUDE_INSTALL_DIR@ by ../../../include and @PACKAGE_MYPKGDATA_INSTALL_DIR@ by ../../../share/mypkg (both transformed to be relative to DESTINATION). A problem I see here (and which

Re: [cmake-developers] Making Config.cmake files easier to write

2012-02-14 Thread Yury G. Kudryashov
for no real gain. A simple API would be write me a basic Config.cmake file for this export set but this would require a few changes in C++ code (i.e., turn export set into an object with properties like DESTINATION, FILE, NAMESPACE, TARGETS). -- Yury G. Kudryashov, mailto: ur...@mccme.ru

Re: [cmake-developers] [PATCH] FindPackageHandleStandardArgs: fix documentation

2012-01-20 Thread Yury G. Kudryashov
Brad King wrote: On 1/19/2012 2:41 PM, Yury G. Kudryashov wrote: State explicitly what CONFIG_MODE argument does. [snip] -# a find_package(... NO_MODULE) call, in this case all the information -# provided by the config-mode of find_package() will be evaluated -# automatically

[cmake-developers] [PATCH] Remove duplicated sentense from docs

2012-01-17 Thread Yury G. Kudryashov
From: Yury G. Kudryashov urkud.ur...@gmail.com install(TARGETS ...) documentation described twice what happens on non-DLL systems. --- Source/cmInstallCommand.h |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/Source/cmInstallCommand.h b/Source/cmInstallCommand.h index