MantisBT - CMake
View Issue Details
0014663CMakeCMakepublic2013-12-22 07:542014-06-02 08:37
Andrius da Costa Ribas 
Stephen Kelly 
highcrashalways
closedno change required 
WindowsWindows8
CMake 2.8.12 
 
0014663: Duplicated include_directories
CMake do not remove duplicates from include_directories.
Have a list of diretories added to include_directories in the main CMakeLists.txt and the same list added to include_directories in a subdir.
This crashes building projects like Calligra due to too long command line (33 times the same include paths passed to -I). I've proposed a workaround for them in https://git.reviewboard.kde.org/r/114611/diff/#index_header [^] .
No tags attached.
Issue History
2013-12-22 07:54Andrius da Costa RibasNew Issue
2013-12-22 09:27Stephen KellyNote Added: 0034853
2013-12-22 09:54Andrius da Costa RibasNote Added: 0034854
2013-12-23 11:14Stephen KellyNote Added: 0034858
2013-12-23 11:14Stephen KellyStatusnew => resolved
2013-12-23 11:14Stephen KellyResolutionopen => no change required
2013-12-23 11:14Stephen KellyAssigned To => Stephen Kelly
2014-06-02 08:37Robert MaynardNote Added: 0036064
2014-06-02 08:37Robert MaynardStatusresolved => closed

Notes
(0034853)
Stephen Kelly   
2013-12-22 09:27   
http://sscce.org/ [^]
(0034854)
Andrius da Costa Ribas   
2013-12-22 09:54   
Just pasting a link to sscce.org with no pointing about what is missing sounds very paradoxical to me.

Short: the description is short - include_directories do not remove duplicates and cause a crash due to long command line.

Self-contained: Building Calligra on Windows fails due to this (example given on "Additional Information" - KOMAIN_INCLUDES contains includes already included in parent directory)

Correct: not applicable here according to the definition in sscce.org

Example: Provided
(0034858)
Stephen Kelly   
2013-12-23 11:14   
No demonstrated problem to confirm.

CMake de-duplicates include directories already.
(0036064)
Robert Maynard   
2014-06-02 08:37   
Closing resolved issues that have not been updated in more than 4 months.