MSVC: Incorrect setting of MSVC64 (Bug #2780)


Added by Leszek Swirski about 12 years ago. Updated over 11 years ago.


Status:Cancelled Start date:
Priority:Normal Due date:
Assignee:Vadim Pisarevsky % Done:

0%

Category:build/install
Target version:-
Affected version:branch 'master' (2.4.9) Operating System:
Difficulty: HW Platform:
Pull request:

Description

MSVC64 is set by checking the value of CMAKE_CL_64 in cmake/OpenCVDetectCXXCompiler.cmake. However, there is a bug in CMake (http://www.cmake.org/Bug/view.php?id=13896) where CMAKE_CL_64 is incorrectly set to 0 for CXX projects.

Currently, the opencv project is declared in CMakeLists.txt as project(opencv CXX C), which is affected by this bug. Until CMake fixes this issue, the workaround is to change CMakeLists.txt to say project(opencv C CXX).


Associated revisions

Revision 8d58b238
Added by Vadim Pisarevsky over 10 years ago

Merge pull request #2780 from hbadino:Feature_3692_2.4

History

Updated by Vadim Pisarevsky about 12 years ago

I can not reproduce the problem. The inserted message(STATUS "CMAKE_CL_64: ${CMAKE_CL_64}") prints "1" with VS2012 x64 and cmake 2.8.10.2.

  • Assignee set to Vadim Pisarevsky
  • Priority changed from High to Normal
  • Category set to build/install

Updated by Leszek Swirski almost 12 years ago

Can you reproduce the problem as described in the cmake issue, with a fresh, trivial project?

Updated by Leszek Swirski almost 12 years ago

Well, CMake confirmed (and fixed) the issue, and I've reproduced it on another machine. Can we change that line in the OpenCV CMakeLists.txt for compatibility with previous CMake versions?

Updated by Leszek Swirski almost 12 years ago

I've submitted a pull request: https://github.com/Itseez/opencv/pull/888

Updated by Andrey Pavlenko over 11 years ago

The problem isn't reproduced when I specify:
X:\Apps\cmake-2.8.10.2-win32-x86\bin\cmake.exe -G "Visual Studio 11 Win64" .

  • Start date deleted (2013-02-07)

Updated by Andrey Pavlenko over 11 years ago

  • Status changed from Open to Cancelled

Also available in: Atom PDF