STR #1123

GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 Home  |  Articles & FAQs  |  Bugs & Features  |  Documentation  |  Download  |  Screenshots  ]
 

Return to Bugs & Features | Roadmap 1.1 ]

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:

Receive EMails Don't Receive EMails

Trouble Report Files:

No files


Trouble Report Comments:


Name/Time/Date Text top right image
 
#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.
 
bottom left image   bottom right image

Return to Bugs & Features ]

 
 

Comments are owned by the poster. All other content is copyright 1998-2021 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.