paceholder / nodeeditor

Qt Node Editor. Dataflow programming framework
BSD 3-Clause "New" or "Revised" License
3.03k stars 814 forks source link

Unable to build when using `vcpkg` toolchain #321

Closed arsdever closed 1 year ago

arsdever commented 2 years ago

The vcpkg toolchain causes infinite recursion.

[main] Building folder: nodeeditor 
[main] Configuring folder: nodeeditor 
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --no-warn-unused-cli -DCMAKE_TOOLCHAIN_FILE=D:/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -DCMAKE_BUILD_TYPE:STRING=Debug "-DCMAKE_C_COMPILER:FILEPATH=C:\Program Files\LLVM\bin\clang.exe" "-DCMAKE_CXX_COMPILER:FILEPATH=C:\Program Files\LLVM\bin\clang++.exe" -Sd:/nodeeditor -Bd:/nodeeditor/build -G Ninja
[cmake] Not searching for unused variables given on the command line.
[cmake] CMake Error at D:/vcpkg/scripts/buildsystems/vcpkg.cmake:746 (set):
[cmake]   Maximum recursion depth of 100 exceeded
[cmake] Call Stack (most recent call first):
[cmake]   D:/vcpkg/scripts/buildsystems/vcpkg.cmake:824 (_find_package)
...
[cmake]   D:/vcpkg/scripts/buildsystems/vcpkg.cmake:824 (_find_package)
[cmake] -- Configuring incomplete, errors occurred!
[cmake] See also "D:/nodeeditor/build/CMakeFiles/CMakeOutput.log".
[cmake]   D:/vcpkg/scripts/buildsystems/vcpkg.cmake:824 (_find_package)
...
[cmake]   D:/vcpkg/scripts/buildsystems/vcpkg.cmake:824 (_find_package)
[cmake]   external/CMakeLists.txt:11 (_find_package)
[cmake]   CMakeLists.txt:47 (find_package)

You have vcpkg installed and configured, and qtbase installed through vcpkg. Then you try to configure the repository with the following argument in the CMake command

cmake -B<builddir> -S<sourcedir> -DCMAKE_TOOLCHAIN_FILE=/path/to/vcpkg/scripts/buildsystems/vcpkg.cmake

The issue is met on a branch v3alpha

paceholder commented 2 years ago

Hello!

Thanks for reporting this issue. Unfortunately I am not an expert in vcpkg -- never got a chance of using it. I am afraid I need some help in resolving this problem

Best, Dmitry

paceholder commented 1 year ago

Fixed in master (v2), will merge the version 3 in the nearest days with all the fixes