Platformio-core: [CLion] CMakeListsUser.txt is not included in CMakeLists.txt

Created on 26 Jun 2019 · 3Comments · Source: platformio/platformio-core

What kind of issue is this?

[x] PlatformIO Core.

If you’ve found a bug, please provide an information below

Configuration

Operating system : Debian 9, kernel 4.9.168-1+deb9u3 x86_64

PlatformIO Version : 4.0.0b3

Python Version : 3.7.3

CLion Version : 2019.1.4

Description of problem

CMakeListsUser.txt is not included in the CMakeLists.txt file thus preventing the use of custom CMake operations.

Steps to Reproduce

Init a new paltoformio project with --ide clion
Add a template main.cpp in src/
Init once more (As per the documentation).
Open in CLion. At this point everything should load fine and CMake will create the cmake-build-debug directory (Default in CLion)
Create the CMakeListsUser.txt file containing message(${CMAKE_VERSION}) at the root of the project
Reload CMake project (CMake tab, reload)

Actual Results

-- Configuring done
-- Generating done

CMakeListsUser.txt is not included.

Expected Results

3.14.3
-- Configuring done
-- Generating done

CMakeListsUser.txt is included.

Additional info

https://github.com/platformio/platformio-core/blob/54921c5dbdc68893fe45649d07d067818c36889b/platformio/ide/tpls/clion/CMakeLists.txt.tpl#L13-L15

From what I can make out, the if(EXISTS CMakeListsUser.txt) resolves relative paths from the cmake-build-debug directory but not include(CMakeListsUser.txt) , which resolves from the root of the project. In the current state, touching a "CMakeListsUser.txt" file in the cmake-build-debug directory makes it work as expected, but this should not be needed.

Proposed fix

Changing if(EXISTS CMakeListsUser.txt) to if(EXISTS ${CMAKE_CURRENT_SOURCE_DIR}/CMakeListsUser.txt) looks like a clean fix to me, better than what I initially proposed in my forum post.

bug

Source