| View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
| 0015220 | CMake | Modules | public | 2014-10-27 12:10 | 2015-04-06 09:07 | ||||
| Reporter | Anton Indrawan | ||||||||
| Assigned To | Brad King | ||||||||
| Priority | normal | Severity | minor | Reproducibility | always | ||||
| Status | closed | Resolution | fixed | ||||||
| Platform | OS | QNX | OS Version | 6.5 | |||||
| Product Version | CMake 3.0 | ||||||||
| Target Version | CMake 3.1 | Fixed in Version | CMake 3.1 | ||||||
| Summary | 0015220: Find_package(Curses) failed for QNX 6.5 and QNX 6.6 | ||||||||
| Description | When using find_package(Curses) for a cmake project compiled with the QNX 6.5 or QNX 6.6 toolchain, cmake threw the following error: CMake Error at /usr/local/share/cmake-3.0/Modules/FindCurses.cmake:139 (CHECK_LIBRARY_EXISTS): Unknown CMake command "CHECK_LIBRARY_EXISTS". Call Stack (most recent call first): TestProgram/curses/CMakeLists.txt:3 (find_package) The same error is reproducible on cmake 3.0.2. | ||||||||
| Steps To Reproduce | To reproduce the issue, you have to have a QNX 6.5 or QNX 6.6 toolchain installed. 1. Please create a simple cmake project which calls "find_package(Curses)" project(test.curses) find_package(Curses) add_executable(${PROJECT_NAME} main.cpp) 2. In the main.cpp, it can just be an empty int main. | ||||||||
| Additional Information | The toolchains I am using are standard QNX 6.5 and QNX 6.6 toolchains downloaded from www.qnx.com. The host OS is Ubuntu 12.04 64-bit. | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files | |||||||||
| Relationships | |
| Relationships |
| Notes | |
|
(0037076) Ben Boeckel (developer) 2014-10-27 13:36 |
Looks like CheckLibraryExists.cmake is included in one if() block while also being used in another if block with a different condition on it. CheckLibraryExists.cmake should probably be included unconditionally. |
|
(0037083) Brad King (manager) 2014-10-28 08:51 |
Please try this patch:diff --git a/Modules/FindCurses.cmake b/Modules/FindCurses.cmake
index 0184c39..6e17411 100644
--- a/Modules/FindCurses.cmake
+++ b/Modules/FindCurses.cmake
@@ -100,6 +100,7 @@ if(CURSES_CURSES_LIBRARY AND CURSES_NEED_NCURSES)
endif()
if(CURSES_USE_NCURSES)
+ include(${CMAKE_CURRENT_LIST_DIR}/CheckLibraryExists.cmake)
get_filename_component(_cursesLibDir "${CURSES_NCURSES_LIBRARY}" PATH)
get_filename_component(_cursesParentDir "${_cursesLibDir}" PATH) |
|
(0037084) Brad King (manager) 2014-10-28 08:58 |
Re 0015220:0037076: Yes, unconditional inclusion is simpler. Done: FindCurses: Include CheckLibraryExists before using it http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=f11f9579 [^] |
|
(0037118) Brad King (manager) 2014-10-31 11:40 |
I've merged this to the 'release' branch for inclusion in the next 3.1 release candidate. |
|
(0038434) Robert Maynard (manager) 2015-04-06 09:07 |
Closing resolved issues that have not been updated in more than 4 months. |
| Notes |
| Issue History | |||
| Date Modified | Username | Field | Change |
| 2014-10-27 12:10 | Anton Indrawan | New Issue | |
| 2014-10-27 13:36 | Ben Boeckel | Note Added: 0037076 | |
| 2014-10-28 08:51 | Brad King | Note Added: 0037083 | |
| 2014-10-28 08:58 | Brad King | Note Added: 0037084 | |
| 2014-10-28 08:58 | Brad King | Assigned To | => Brad King |
| 2014-10-28 08:58 | Brad King | Status | new => assigned |
| 2014-10-28 08:58 | Brad King | Target Version | => CMake 3.2 |
| 2014-10-31 11:40 | Brad King | Note Added: 0037118 | |
| 2014-10-31 11:40 | Brad King | Status | assigned => resolved |
| 2014-10-31 11:40 | Brad King | Resolution | open => fixed |
| 2014-10-31 11:40 | Brad King | Fixed in Version | => CMake 3.1 |
| 2014-10-31 11:40 | Brad King | Target Version | CMake 3.2 => CMake 3.1 |
| 2015-04-06 09:07 | Robert Maynard | Note Added: 0038434 | |
| 2015-04-06 09:07 | Robert Maynard | Status | resolved => closed |
| Issue History |
| Copyright © 2000 - 2018 MantisBT Team |