CMake error if WITH_CUDA=ON (Bug #711)

Added by Martin Baeuml over 14 years ago. Updated over 14 years ago.

Status:Done Start date:
Priority:High Due date:
Assignee:Anatoly Baksheev % Done:

0%
Category:build/install
Target version:-
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

When setting WITH_CUDA to ON, CMake fails with the following error message:

CMake Error at /cvhci/distlibs/cmake-2.8.1-Linux-i386/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:70 (MESSAGE):
   REQUIRED_VARS (missing: VERSION_VAR)
 Call Stack (most recent call first):
   modules/gpu/FindNPP.cmake:107 (find_package_handle_standard_args)
   modules/gpu/CMakeLists.txt:38 (find_package)

Associated revisions

Revision a7d979fb
Added by Martin Baeum over 14 years ago

remove spurious VERSION_VAR (fixes #711)

Revision 7ff68d5e
Added by Andrey Kamaev almost 12 years ago

Merge pull request #711 from taka-no-me:move_keypoint_dmatch

History

#1
Updated by Martin Baeuml over 14 years ago

  • Status changed from Open to Done
  • (deleted custom field) set to fixed

#2
Updated by Anatoly Baksheev over 14 years ago

I restored VERSION_VAR. Please update CMake, do not delete this!!!

#3
Updated by Martin Baeuml over 14 years ago

Sorry about the this. However, I have a pretty recent CMake (2.8.1) which I guess many other users also have, so maybe there is a workaround that makes compiling possible even without a bleeding edge CMake (?).

If not, there should be a CMAKE_MINIMUM_REQUIRED(VERSION 2.8.3) somewhere ...

  • Status changed from Done to Cancelled
  • (deleted custom field) deleted (fixed)

#4
Updated by Anatoly Baksheev over 14 years ago

No problem.

The construction brings version check for NPP and of found not sufficient a message will be printed.We want to make user to update NPP, not asking us by email about compile-time errors.

Workaround is to eliminate FindPackageHandleStandartArg usage and replace with our own code. And threrefore to lose universality or to bring a lot of code here. I don't think there is a big problem to update cmake.

String CMAKE_MINIMUM_REQUIRED(VERSION 2.8.3) will be added with next commit today. Thanks.

#5
Updated by Anatoly Baksheev over 14 years ago

Reverted CMAKE_MINIMUM_REQUIRED to 2.8.0. And temporary commented VERSION_VAR. Will wait until 2.8.3 can be installed via apt-get. Addded version check in code instead.

#6
Updated by Vadim Pisarevsky over 14 years ago

Anatoly, since you are working on the ticket, I assign it to you

  • Status deleted (Cancelled)

#7
Updated by Anatoly Baksheev over 14 years ago

There is no reason to leave this ticket as open.

  • Status set to Done
  • (deleted custom field) set to fixed

Also available in: Atom PDF