| View Issue Details [ Jump to Notes ] | [ Print ] |
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0013665 | CMake | CMake | public | 2012-11-08 17:01 | 2014-02-10 10:44 |
|
| Reporter | mwoehlke | |
| Assigned To | Stephen Kelly | |
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | no change required | |
| Platform | x86_64 | OS | Fedora | OS Version | 17 |
| Product Version | CMake 2.8.10 | |
| Target Version | | Fixed in Version | | |
|
| Summary | 0013665: include_directories(SYSTEM) should promote |
| Description | In some instances (possibly related to CMake version), include_directories(SYSTEM ${foo}) has no effect. |
| Steps To Reproduce | include_directories(${foo})
include_directories(SYSTEM ${foo}) |
| Additional Information | It appears that CMake ignores an include_directories(SYSTEM) for include directories that already exist. This is more likely in large projects where e.g. a top-level CMake file (or even a UseFoo.cmake) includes directories without SYSTEM, and a leaf CMakeLists.txt does an include_directories(SYSTEM) of the same directories.
I think it would be better following the expected behavior of include_directories if, rather than ignore the directive if the directory is already in include_directories, to 'promote' it to SYSTEM (probably without changing the order). |
| Tags | No tags attached. |
|
| Attached Files | |
|