[cmake-developers] Nightly Binary not building on windows

[David Cole]

On Tue, Feb 28, 2012 at 1:25 PM, Bill Hoffman <bill.hoffman at kitware.com> wrote:
> On 2/28/2012 1:14 PM, Eric Noulard wrote:
>>
>> 2012/2/28 Bill Hoffman<bill.hoffman at kitware.com>:
>>>
>>> The nightly binary for CMake is not building because of a test failure:
>>>
>>> http://open.cdash.org/testDetails.php?test=136868750&build=2046004
>>>
>>>
>>> CMake Error at CMakeLists.txt:52 (message):
>>>  Subversion_VERSION_SVN has unexpected content
>>> Call Stack (most recent call first):
>>>  CMakeLists.txt:74 (check_version_string)
>>>
>>> The following tests FAILED:
>>>        185 - CMakeOnly.AllFindModules (Failed)
>>
>>
>> This commit
>> cffce2ffb366cd5c598f56e8f0789124a6d1b7a9
>> added at least this
>> check_version_string(SUBVERSION Subversion_VERSION_SVN)
>>
>> Subversion_VERSION_SVN seems empty (from the error message)?
>>
>> The module is named "FindSubversion.cmake" make be
>>
>> check_version_string(SUBVERSION Subversion_VERSION_SVN)
>>
>> should simply be:
>>
>> check_version_string(Subversion Subversion_VERSION_SVN)
>>
>> upper/lower case trouble?
>>
>> Mixed-case leads to trouble since (from FindSubversion.cmake) we have:
>>
>> #  Subversion_FOUND - true if the command line client was found
>> #  SUBVERSION_FOUND - same as Subversion_FOUND, set for compatiblity
>> reasons
>>
>
> OK, the problem is that svn is not working on the machine:
>
> $ /cygdrive/c/Program\ Files\ \(x86\)/Subversion/svn info
> svn: Can't determine the user's config path
>
>
> However, this should not be an error.  That is the point of this test. Even
> if something is wrong, it should not error out.  It should continue unless
> it is required.  Perhaps the check_version_string function should be made
> aware that it should only error out if REQUIRED is set.
>
>
> -Bill
>
>
> --
>
> 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


Wait for Eike to reply here. He's been requiring version variables to
be set if the package is found in this test. I'm sure he'll either
patch this up so it works or exclude svn from the version variable
test depending on how hard it is to detect this situation...


Thx,
David