| [ Return to Bugs & Features | Roadmap 1.1 | SVN ⇄ GIT ]
STR #1123
Application: | FLTK Library |
Status: | 1 - Closed w/Resolution |
Priority: | 3 - Moderate, e.g. unable to compile the software |
Scope: | 3 - Applies to all machines and operating systems |
Subsystem: | Build Files |
Summary: | CMakeLists.txt file broke FLTK_INCLUDE_DIRS |
Version: | 1.1-current |
Created By: | luisibanez |
Assigned To: | matt |
Fix Version: | 1.1.7 (SVN: v4715) |
Update Notification: | |
Trouble Report Files:
No files
Trouble Report Comments:
|
#1 | luisibanez 07:52 Dec 14, 2005 |
| It seems that recent changes to the CMakeLists.txt file in the 1.1 branch broke the settings for FLTK_INCLUDE_DIRS.
The directory got set to
SET(FLTK_INCLUDE_DIRS "${FLTK_BINARY_DIR}/FL")
but it should be only:
SET(FLTK_INCLUDE_DIRS "${FLTK_BINARY_DIR}/")
because all the #includes for FLTK already have the "Fl/" subdirectory explicitly listed.
We have fixed this in our local subversion checkout. Please find the svn diff at the end of this message.
--
We are maintaining at Kitware.com a Dashboard for FLTK in multiple platforms. You can find the Nightly builds at
http://public.kitware.com/dashboard.php?name=fltk
--
Thanks for maintaing FLTK, It is such a great library !!
Regards,
Luis
--
Luis Ibanez, Ph.D. Senior Research Engineer KITWARE Inc.
-------------------------------------- Index: CMakeLists.txt =================================================================== --- CMakeLists.txt (revision 4702) +++ CMakeLists.txt (working copy) @@ -367,7 +367,7 @@ SET(FLTK_EXECUTABLE_DIRS ${EXECUTABLE_OUTPUT_PATH}) SET(FLTK_LIBRARY_DIRS ${LIBRARY_OUTPUT_PATH}) SET(FLTK_USE_FILE ${FLTK_SOURCE_DIR}/CMake/FLTKUse.cmake) -SET(FLTK_INCLUDE_DIRS "${FLTK_BINARY_DIR}/FL") +SET(FLTK_INCLUDE_DIRS "${FLTK_BINARY_DIR}/") SET(FLTK_BUILD_SETTINGS_FILE ${FLTK_BINARY_DIR}/FLTKBuildSettings.cmake) SET(prefix "${FLTK_BINARY_DIR}") SET(exec_prefix "${prefix}") @@ -388,7 +388,7 @@ SET(FLTK_EXECUTABLE_DIRS "${CMAKE_INSTALL_PREFIX}/bin") SET(FLTK_LIBRARY_DIRS "${CMAKE_INSTALL_PREFIX}/lib") SET(FLTK_USE_FILE "${CMAKE_INSTALL_PREFIX}/lib/FLTK-${FLTK_VERSION}/FLTKUse.cmake") -SET(FLTK_INCLUDE_DIRS "${CMAKE_INSTALL_PREFIX}/FL") +SET(FLTK_INCLUDE_DIRS "${CMAKE_INSTALL_PREFIX}/") SET(FLTK_BUILD_SETTINGS_FILE ${CMAKE_INSTALL_PREFIX}/lib/FLTK-${FLTK_VERSION}/FLTKBuildSettings.cmake) SET(prefix "${CMAKE_INSTALL_PREFIX}") SET(exec_prefix "${prefix}") | |
|
#2 | matt 07:23 Dec 16, 2005 |
| Fixed in Subversion repository.
I don't use CMake myself. But since you guys actually wrote CMake, I assum that your patch is A OK. We are very happy that you guys like our library. | |
[ Return to Bugs & Features ]
|
| |