MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012466CMakeModulespublic2011-09-19 09:252012-07-09 06:52
Reporterogronom 
Assigned ToRolf Eike Beer 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake 2.8.5 
Target VersionCMake 2.8.8Fixed in VersionCMake 2.8.8 
Summary0012466: FindGLUT does not honor REQUIRED
DescriptionWith no glut installed cmake does not stop build phase in spite of REQUIRED.
Steps To Reproduce#CMskeLists.txt
cmake_minimum_required(VERSION 2.8)
find_package(GLUT REQUIRED)
if (NOT GLUT_FOUND)
    message("Possible Bug???")
endif (NOT GLUT_FOUND)
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch 0001-FindGLUT-honor-REQUIRED-12466.patch (1,835) 2012-02-08 14:06
https://public.kitware.com/Bug/file/4209/0001-FindGLUT-honor-REQUIRED-12466.patch
Issue History
Date ModifiedUsernameFieldChange
2011-09-19 09:25ogronomNew Issue
2011-09-19 09:57Brad KingNote Added: 0027446
2012-02-08 14:05Rolf Eike BeerNote Added: 0028527
2012-02-08 14:06Rolf Eike BeerFile Added: 0001-FindGLUT-honor-REQUIRED-12466.patch
2012-02-09 08:58Brad KingAssigned To => Rolf Eike Beer
2012-02-09 08:58Brad KingStatusnew => assigned
2012-02-15 15:57Rolf Eike BeerNote Added: 0028595
2012-02-15 15:57Rolf Eike BeerStatusassigned => resolved
2012-02-15 15:57Rolf Eike BeerResolutionopen => fixed
2012-02-15 15:57Rolf Eike BeerFixed in Version => CMake 2.8.8
2012-02-15 15:57Rolf Eike BeerTarget Version => CMake 2.8.8
2012-07-09 06:52David ColeNote Added: 0029975
2012-07-09 06:52David ColeStatusresolved => closed

Notes
(0027446)
Brad King   
2011-09-19 09:57   
The FindGLUT module predates the find_package command's REQUIRED infrastructure. There is no module maintainer for it:

  http://www.cmake.org/Wiki/CMake:Module_Maintainers [^]

so it was never updated.
(0028527)
Rolf Eike Beer   
2012-02-08 14:05   
Please test the attached patch if it works for you with and without GLUT.
(0028595)
Rolf Eike Beer   
2012-02-15 15:57   
Fix pushed to next:

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=14dadbde809d5091de44ef880376f69f6be2e23e [^]
(0029975)
David Cole   
2012-07-09 06:52   
Closing resolved issues that have not been updated in more than 4 months.