Re: [cmake-developers] CMake is slow for project with hundred target and one command with large number of dependencies

2013-07-15 Thread Nicolas Desprès
On Sat, Jul 13, 2013 at 10:21 AM, Nicolas Desprès nicolas.desp...@gmail.com wrote: On Fri, Jul 12, 2013 at 11:35 PM, Stephen Kelly steve...@gmail.comwrote: Nicolas Desprès wrote: I have pushed my work so far on my github clone of CMake.

Re: [cmake-developers] CMake is slow for project with hundred target and one command with large number of dependencies

2013-07-15 Thread Stephen Kelly
Nicolas Desprès wrote: I have pushed my work so far on my github clone of CMake. https://github.com/nicolasdespres/CMake/tree/topic/large-deps-perf Thanks. The bottleneck seems to be in cmTargetTraceDependencies::FollowName as it follows each entry in the LARGE_LIST for each of the 100

Re: [cmake-developers] CMake is slow for project with hundred target and one command with large number of dependencies

2013-07-15 Thread Nicolas Desprès
On Mon, Jul 15, 2013 at 11:53 AM, Stephen Kelly steve...@gmail.com wrote: Nicolas Desprès wrote: I have pushed my work so far on my github clone of CMake. https://github.com/nicolasdespres/CMake/tree/topic/large-deps-perf Thanks. The bottleneck seems to be in

Re: [cmake-developers] CMake is slow for project with hundred target and one command with large number of dependencies

2013-07-15 Thread Nicolas Desprès
On Mon, Jul 15, 2013 at 12:45 PM, Nicolas Desprès nicolas.desp...@gmail.com wrote: On Mon, Jul 15, 2013 at 11:53 AM, Stephen Kelly steve...@gmail.comwrote: Nicolas Desprès wrote: I have pushed my work so far on my github clone of CMake.

Re: [cmake-developers] CMake is slow for project with hundred target and one command with large number of dependencies

2013-07-15 Thread Stephen Kelly
Nicolas Desprès wrote: Yes that one or cmMakefile::GetSourceFileWithOutput(char const*). I can send you a gzipped of my callgrind data off-list if you want (it weights 534K). If we had a map associating each input to output, we could maybe have better performance. WDYT? Perhaps. I can

Re: [cmake-developers] Working towards Windows Phone 8 support

2013-07-15 Thread Brad King
On 07/12/2013 07:58 AM, Paul Annetts wrote: I'm interested in contributing towards Windows Phone 8 support in CMAKE, i.e. http://public.kitware.com/Bug/view.php?id=13791 Thanks for working on this. Consider drawing attention to your work by posting in the issue tracker too. I haven't

Re: [cmake-developers] Why does this work - linking against a library before it has been defined ?

2013-07-15 Thread Brad King
On 07/11/2013 03:52 PM, Alexander Neundorf wrote: target_link_libraries(kf5-config ${KDE4_KDECORE_LIBS} kde4support) add_subdirectory(lib) The thing is, the library kde4support is built inside the lib subdirectory, which is added after using kde4support in target_link_libraries(). I

Re: [cmake-developers] CMAKE_SYSROOT vs CMAKE_OSX_SYSROOT

2013-07-15 Thread Brad King
On 07/12/2013 09:47 AM, Stephen Kelly wrote: Brad King wrote: On 07/11/2013 11:08 AM, Brad King wrote: Can anyone else think of complications? Side note: If we cannot resolve this soon we should revert CMAKE_SYSROOT before 2.8.12 to leave room for the two to be combined later. I've

Re: [cmake-developers] CMAKE_INSTALL_PREFIX vs CMAKE_SYSROOT ?

2013-07-15 Thread Brad King
On 07/14/2013 10:13 AM, Samuel Martin wrote: I'd like to give a point of view of a user. Great, thanks for your input. To stick back to this thread, I have not used the CMAKE_SYSROOT variable yet. But imho, it should only affect the compiler flags, not the install rules because the given

Re: [cmake-developers] CMAKE_INSTALL_PREFIX vs CMAKE_SYSROOT ?

2013-07-15 Thread Stephen Kelly
Brad King wrote: On 07/04/2013 04:29 AM, Stephen Kelly wrote: Goals: * Install to /opt/kf5 on the target * Install to $HOME/dev/kf5 on the host Steve, why do we need to have the host location not end in the target's install prefix? For me, it's because I install things to the same

[cmake-developers] install-interface-includes topic

2013-07-15 Thread Brad King
Steve, From the documentation of the feature: + The INCLUDES DESTINATION specifies a list of directories which will + be added to the INTERFACE_INCLUDE_DIRECTORIES of the targets when + exported by install(EXPORT). If a relative path is specified, it is + treated as

[cmake-developers] [CMake 0014291]: add_custom_command() DEPENDS does not work with the Xcode generator

2013-07-15 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://www.cmake.org/Bug/view.php?id=14291 == Reported By:Paul Smith Assigned To:

Re: [cmake-developers] Working towards Windows Phone 8 support

2013-07-15 Thread Paul Annetts
Thanks for the response Brad. Comments inline below... Paul. Date: Mon, 15 Jul 2013 08:38:11 -0400 From: brad.k...@kitware.com To: cmake-developers@cmake.org Subject: Re: [cmake-developers] Working towards Windows Phone 8 support On 07/12/2013 07:58 AM, Paul Annetts wrote: I'm interested

[cmake-developers] [CMake 0014292]: CMake does not add QT_NO_DEBUG definition for non-debug builds

2013-07-15 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=14292 == Reported By:Nikita Krupen'ko Assigned To:

Re: [cmake-developers] CMake is slow for project with hundred target and one command with large number of dependencies

2013-07-15 Thread Nicolas Desprès
On Mon, Jul 15, 2013 at 1:51 PM, Stephen Kelly steve...@gmail.com wrote: Nicolas Desprès wrote: Yes that one or cmMakefile::GetSourceFileWithOutput(char const*). I can send you a gzipped of my callgrind data off-list if you want (it weights 534K). If we had a map associating each

Re: [cmake-developers] install-interface-includes topic

2013-07-15 Thread Brad King
On 7/15/2013 11:25 AM, Stephen Kelly wrote: The solution I think is to add the specified include directories to the cmTargetExport instance which gets created when using the EXPORT, and to append the directories only when generating. Yes, certainly the INCLUDES DESTINATION needs to be

Re: [cmake-developers] Working towards Windows Phone 8 support

2013-07-15 Thread Brad King
On 7/15/2013 11:08 AM, Paul Annetts wrote: Yes. There are only 2 possible architectures for WP8, it isn’t WinCE [snip] The extension of the VS10 target generator to support this is the largest change in the patch. If possible, please factor that out as a separate patch independent of the

Re: [cmake-developers] install-interface-includes topic

2013-07-15 Thread Stephen Kelly
Brad King wrote: On 7/15/2013 11:25 AM, Stephen Kelly wrote: The solution I think is to add the specified include directories to the cmTargetExport instance which gets created when using the EXPORT, and to append the directories only when generating. Yes, certainly the INCLUDES DESTINATION

[cmake-developers] compile-defs-debugging topic

2013-07-15 Thread Brad King
Steve, I merged this topic, consisting of commits a18e9586..d7dd0108, to master this morning. However, when I use the VS generators on Windows CMake now crashes because something clobbers the cmMakefile::SourceGroups member. This didn't crash when the topic was in next. Does it depend on

Re: [cmake-developers] Working towards Windows Phone 8 support

2013-07-15 Thread Paul Annetts
Sure - that should separate out well into two parts (Win32/ARM architecture and Windows Phone). The way I'm currently specifying the multiple architectures is too hard-coded to be generally useful (I'm just checking the generator type), so I'll see if I can tidy that up first before submitting

[cmake-developers] [CMake 0014294]: Command-line options overwritten by re-run

2013-07-15 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=14294 == Reported By:Greg Coombe Assigned To: