MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0014067CMakeModulespublic2013-04-09 10:092016-06-10 14:31
ReporterChristian Bühler 
Assigned ToKitware Robot 
PrioritynormalSeverityblockReproducibilityalways
StatusclosedResolutionmoved 
PlatformWindowsOSOS Version
Product VersionCMake 2.8.10.2 
Target VersionFixed in Version 
Summary0014067: FindCoin3D.cmake looks for coin2 instead of coin3
Description    find_library(COIN3D_LIBRARY_DEBUG coin2d
      "[HKEY_LOCAL_MACHINE\\SOFTWARE\\SIM\\Coin3D\\2;Installation Path]/lib"
    )

    find_library(COIN3D_LIBRARY_RELEASE coin2
      "[HKEY_LOCAL_MACHINE\\SOFTWARE\\SIM\\Coin3D\\2;Installation Path]/lib"
    )

but should be

    find_library(COIN3D_LIBRARY_DEBUG coin3d
      "[HKEY_LOCAL_MACHINE\\SOFTWARE\\SIM\\Coin3D\\2;Installation Path]/lib"
    )

    find_library(COIN3D_LIBRARY_RELEASE coin3
      "[HKEY_LOCAL_MACHINE\\SOFTWARE\\SIM\\Coin3D\\2;Installation Path]/lib"
    )
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-04-09 10:09Christian BühlerNew Issue
2013-04-09 10:27Brad KingAssigned To => Matt Leotta
2013-04-09 10:27Brad KingStatusnew => assigned
2013-04-09 11:38Brad KingNote Added: 0032785
2013-04-09 11:38Brad KingAssigned ToMatt Leotta =>
2013-04-09 11:38Brad KingStatusassigned => backlog
2013-04-09 11:39Brad KingNote Added: 0032786
2013-04-09 12:02Christian BühlerNote Added: 0032788
2016-06-10 14:28Kitware RobotNote Added: 0042263
2016-06-10 14:28Kitware RobotStatusbacklog => resolved
2016-06-10 14:28Kitware RobotResolutionopen => moved
2016-06-10 14:28Kitware RobotAssigned To => Kitware Robot
2016-06-10 14:31Kitware RobotStatusresolved => closed

Notes
(0032785)
Brad King   
2013-04-09 11:38   
This module currently has no maintainer:

 http://www.cmake.org/Wiki/CMake:Module_Maintainers [^]
(0032786)
Brad King   
2013-04-09 11:39   
Can you explain why the module might have needed to search for coin2 at one time in the past?
(0032788)
Christian Bühler   
2013-04-09 12:02   
While searching for an answer to this, I also started wondering where this registry keys come from. I can only guess, but I think both of these are relicts from the Version 2 releases of Coin3D.
But it is hard to prove, as in the meanwhile, only version 3 releases are available (https://bitbucket.org/Coin3D/coin/downloads [^]) and the old website completely vanished in favour of the BitBucket page.

Considering this, I would suggest to remove this module completely because obviously nobody uses it on Windows, and on Linux it can be replaced by two simple lines:

    find_path(COIN3D_INCLUDE_DIRS Inventor/So.h)
    find_library(COIN3D_LIBRARIES Coin)

Also, there is no more an installer for Windows which would justify using the registry keys.
(0042263)
Kitware Robot   
2016-06-10 14:28   
Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current CMake Issues page linked in the banner at the top of this page.