UseEcos: Append C and CXX flags instead of overwriting them (!2684) · Merge requests · CMake / CMake

antmak requested to merge antmak/cmake:patch-1 into master Nov 28, 2018

I was learning how to set CMAKE_C_FLAGS in different configurations, including cross-compiling toolchains, when I noticed some odd code in the UseEcos module. I'm not 100% sure about change, but maybe you could review this. Thanks. Feel free to throw out it :)

(from github https://github.com/Kitware/CMake/pull/324) I did not use string(APPEND) here because there are many places with set(CMAKE_C_FLAGS) in the code. But I'll keep in mind :)

Topic-rename: UseEcos-append-flags

Edited Dec 06, 2018 by Brad King

UseEcos: Append C and CXX flags instead of overwriting them

Merge request reports