Re: [cmake-developers] I: I want to become a maintainer of several modules.

2012-10-05 Thread Alan W. Irwin
On 2012-10-05 13:06-0400 Brad King wrote: The output parsing can be a bit simpler if you use the CMAKE_MATCH_1 variable result from the "if(... MATCHES ...)" command. It is mentioned at the bottom of the regex documentation in string(): http://www.cmake.org/cmake/help/v2.8.9/cmake.html#command

[cmake-developers] [CMake 0013576]: Empty CMAKE_BUILD_TYPE in VS2012

2012-10-05 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=13576 == Reported By:Anders Elfgren Assigned To:

[cmake-developers] [CMake 0013575]: No print messages after PROJECT()

2012-10-05 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=13575 == Reported By:Krzysiek Assigned To:

Re: [cmake-developers] patch for cmake that adds a new timestamp command

2012-10-05 Thread Nils Gladitz
I've updated the patch (attached) to return empty string on failure. I also tried to use the time definitions from global namespace rather than std::. Nils On 10/05/2012 09:17 PM, Brad King wrote: On 10/05/2012 02:53 PM, Nils Gladitz wrote: My initial thought was that "NOTFOUND" would be a g

[cmake-developers] [CMake 0013574]: Ninja: no transitive dependency for imported libraries

2012-10-05 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=13574 == Reported By:Peter Kuemmel Assigned To:

Re: [cmake-developers] patch for cmake that adds a new timestamp command

2012-10-05 Thread Brad King
On 10/05/2012 02:53 PM, Nils Gladitz wrote: > My initial thought was that "NOTFOUND" would be a good idea since it > evaluates to false and the get_*_property commands also seem to use it. I wrote the more general "get_property" command to replace those and the newer command uses empty string rat

Re: [cmake-developers] patch for cmake that adds a new timestamp command

2012-10-05 Thread Nils Gladitz
My initial thought was that "NOTFOUND" would be a good idea since it evaluates to false and the get_*_property commands also seem to use it. Looking at the documentation for "if()" again only "-NOTFOUND" as a suffix should evaluate to false though (I assume the documentation is incomplete here?)

Re: [cmake-developers] patch for cmake that adds a new timestamp command

2012-10-05 Thread Brad King
On 09/29/2012 05:47 AM, Nils Gladitz wrote: > I've updated the patch (attached) to include a "TIMESTAMP" sub-command > for "file" as well (I hope indentation is also fixed). Thanks. Use of std::time_t and std::tm will not compile on all of CMake's platforms. Other code already uses "#include ",

Re: [cmake-developers] I: I want to become a maintainer of several modules.

2012-10-05 Thread Brad King
On 10/05/2012 08:29 AM, Aleksey Avdeev wrote: > I modified the proposed modules. Thanks for working on them. Let's start with FindIcotool since it is pretty simple and doesn't depend on the other modules. After that we can move on to the others. Please proceed with step 5 of the new maintainer

[cmake-developers] [CMake 0013573]: RPATH not supported when cross compiling for FreeBSD

2012-10-05 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://www.cmake.org/Bug/view.php?id=13573 == Reported By:muhkuh Assigned To: =

Re: [cmake-developers] Expression unit test

2012-10-05 Thread Stephen Kelly
Peter Kümmel wrote: > A unit tests for expressions is broken for ninja: > > http://open.cdash.org/testDetails.php?test=161505814&build=2596684 > > > ninja prints the wrong parameter as it is used in the cmake file, > > include_directories("$") > > but the test wants the $ escaped one: >

Re: [cmake-developers] I: I want to become a maintainer of several modules.

2012-10-05 Thread Aleksey Avdeev
12.07.2012 18:41, Brad King пишет: > On 07/10/2012 06:12 PM, Aleksey Avdeev wrote: >> I maintainer of ALT Linux Team (see >> ). Packaging italc2 (see >> ), I made a few modules to >> CMake. I think they will be useful

[cmake-developers] Expression unit test

2012-10-05 Thread Peter Kümmel
A unit tests for expressions is broken for ninja: http://open.cdash.org/testDetails.php?test=161505814&build=2596684 ninja prints the wrong parameter as it is used in the cmake file, include_directories("$") but the test wants the $ escaped one: Expected stderr to match: expect-err>