View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0011678CMakeCMakepublic2011-01-10 16:312011-01-10 16:56
ReporterCuran 
Assigned ToBrad King 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionno change required 
PlatformLinuxOSDebianOS Version6.0+
Product VersionCMake-2-8 
Target VersionFixed in Version 
Summary0011678: FindVTK.cmake fails with additional version requirement.
DescriptionMichael Hanke reported to Debian the bug #600889, stating that FindVTK.cmake fails to find a VTK version if a specific version requirement was given. For Debian this will be fixed in the next upload, the latest version of the patch can be found at <http://git.debian.org/?p=collab-maint/cmake.git;a=blob;f=debian/patches/FindVTK_fix_%23600889.diff;hb=HEAD>, [^] the current version is attached to this report.
Steps To ReproduceUse (e.g.)

FIND_PACKAGE(VTK 5.4 REQUIRED)

instead of

FIND_PACKAGE(VTK REQUIRED)
Additional InformationThe bug was originally reported to Debian as bug #600889, you can find the bug report at <http://bugs.debian.org/600889>. [^]
TagsNo tags attached.
Attached Filesdiff file icon FindVTK_fix_#600889.diff [^] (1,827 bytes) 2011-01-10 16:31 [Show Content]

 Relationships

  Notes
(0024561)
Brad King (manager)
2011-01-10 16:53

The proposed fix is incorrect. The entire block of code around it is specifically looking for VTK 4.0 and nothing more recent. It starts with:

  # Special search for VTK 4.0.

and the case reported in the bug is intentionally commented as

  # We found the wrong version. Pretend we did not find it.

The *real* search takes place before that entire block begins:

  # Use the Config mode of the find_package() command to find VTKConfig.
  # If this succeeds (possibly because VTK_DIR is already set), the
  # command will have already loaded VTKConfig.cmake and set VTK_FOUND.
  IF(NOT _VTK_40_ONLY AND NOT VTK_FOUND)
    FIND_PACKAGE(VTK QUIET NO_MODULE)
  ENDIF()

and is actually implemented in C++ in the find_package() command.

This is actually a VTK issue. It does not provide a package version file (VTKConfigVersion.cmake) file next to its package configuration file (VTKConfig.cmake). Debian can fix this for its VTK packages by adding such a file.
(0024562)
Brad King (manager)
2011-01-10 16:56

No change is required to CMake. Please file this as a VTK issue.

 Issue History
Date Modified Username Field Change
2011-01-10 16:31 Curan New Issue
2011-01-10 16:31 Curan File Added: FindVTK_fix_#600889.diff
2011-01-10 16:53 Brad King Note Added: 0024561
2011-01-10 16:53 Brad King Assigned To => Brad King
2011-01-10 16:53 Brad King Status new => assigned
2011-01-10 16:56 Brad King Note Added: 0024562
2011-01-10 16:56 Brad King Status assigned => closed
2011-01-10 16:56 Brad King Resolution open => no change required


Copyright © 2000 - 2018 MantisBT Team