Add Clion code insight features for PIO "lib_extra_dirs" option

[Copper-Bot]

Configuration

Operating system: Windows 10 64bits

PlatformIO Version (platformio --version): 4.3.1

Clion Version: 2019.3.5

Description of the bug/request

Hi !

When "lib_extra_dirs" option is used on platformio.ini, and when the project is init for Clion, PIO does not add the additional lib path at the end of the CMakeListsPrivate.txt (the path is missing in the "FILE(GLOB_RECURSE SRC_LIST .... )" command).

PIO work fine, but Clion does not activate code insight features for any source files in the additional lib path, unless it is manually added to the CMakeListsPrivate.txt.

Would it be please possible to add this to PIO Clion project files generation ?

[valeros]

Hi @Copper-Bot ! Strange, but I cannot reproduce the issue. I added a new folder to lib_extra_dirs and all libraries from that folder are added to CMakeListsPrivate.txt. Am I missing something?

[Copper-Bot]

Hi @valeros ! Hum, don't you have a warning when opening any .c/.cpp library source file with clion, that they are not part of the project ? Or maybe it is because I am using multiple environments with different "lib_extra_dirs" for each ?

Here is an example: test_folder.zip

If you init the project with Clion, any headers from "lib_extra_dirs" will be part of the clion project (code insight ok), but not their associated .c/.cpp files.

[Copper-Bot]

Hi again,

First thanks you for the PR ! I just switch to the dev version of PIO (4.3.2a2) and re-init my project and ... it doesn't work completely for me haha

It works fine for the first environment, but not the second. It seems that the CMakeListsPrivate.txt does the glob_recurse only for the first env but not the others.

Using the same test_folder.zip project above, I edit the end of the new generated CMakeListsPrivate.txt like this to make it lib_extra_dirs "environment dependent" when calling FILE (GLOB_RECURSE ...) :

...

FILE(GLOB_RECURSE SRC_LIST
    ${CMAKE_CURRENT_LIST_DIR}/src/*.*
    ${CMAKE_CURRENT_LIST_DIR}/lib/*.*
)

if (CMAKE_BUILD_TYPE MATCHES "test_boardone")
    FILE(GLOB_RECURSE SRC_LIST_test_boardone
        ${CMAKE_CURRENT_LIST_DIR}/ext_lib/boardOne/*.*
        ${CMAKE_CURRENT_LIST_DIR}/.pio/libdeps/test_boardone/*.*
    )
    SET(SRC_LIST ${SRC_LIST} ${SRC_LIST_test_boardone})
endif()
if (CMAKE_BUILD_TYPE MATCHES "test_boardtwo")
    FILE(GLOB_RECURSE SRC_LIST_test_boardtwo
        ${CMAKE_CURRENT_LIST_DIR}/ext_lib/boardTwo/*.*
        ${CMAKE_CURRENT_LIST_DIR}/.pio/libdeps/test_boardtwo/*.*
    )
    SET(SRC_LIST ${SRC_LIST} ${SRC_LIST_test_boardtwo})
endif()

I don't know very well Cmake so I don't know if it is a viable solution or not, but it seems to work for Clion.

[valeros]

Strange, but I cannot reproduce the issue with the latest platformio-core. It works fine for both environments. Could you please elaborate on It doesn't work completely for me ?
Intellisense doesn't work or CLion complains that files don't belong to the project?

[Copper-Bot]

Clion complains for the .cpp file of the lib (not the header) :

This occurs only if the 'lib_extra_dirs' is not the same as the first environment.

[valeros]

OK, now I see what you mean, I just pushed a small fix, could you please update PlatformIO to the latest from development branch and try again?

[Copper-Bot]

It works like a charm ! Thank you very much !