Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CMake install instructions are broken #70

Open
Spongman opened this issue Nov 19, 2023 · 2 comments
Open

CMake install instructions are broken #70

Spongman opened this issue Nov 19, 2023 · 2 comments

Comments

@Spongman
Copy link

Spongman commented Nov 19, 2023

https://naios.github.io/continuable/installation.html

$ mkdir test; cd test

$ git init
Initialized empty Git repository in /home/user/test/.git/

$ git submodule add https://github.com/Naios/continuable.git
Cloning into '/home/user/test/continuable'...
remote: Enumerating objects: 7442, done.
remote: Counting objects: 100% (308/308), done.
remote: Compressing objects: 100% (138/138), done.
remote: Total 7442 (delta 156), reused 253 (delta 151), pack-reused 7134
Receiving objects: 100% (7442/7442), 1.90 MiB | 8.09 MiB/s, done.
Resolving deltas: 100% (4954/4954), done.

$ ls continuable/
CMakeLists.txt  Findcontinuable.cmake  LICENSE.txt  Readme.md  appveyor.yml  cmake  conanfile.py  dep  doc  examples  include  test  tools

$ cat >> CMakeLists.txt
cmake_minimum_required(VERSION 3.13)
project(foo)    
add_subdirectory(continuable)
find_package(continuable REQUIRED)
add_executable(foo)
target_link_libraries(foo continuable)

$ cmake -S . -B build
CMake Error at continuable/CMakeLists.txt:90 (find_package):
  By not providing "Findfunction2.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "function2", but CMake did not find one.

  Could not find a package configuration file provided by "function2"
  (requested version 4) with any of the following names:

    function2Config.cmake
    function2-config.cmake

  Add the installation prefix of "function2" to CMAKE_PREFIX_PATH or set
  "function2_DIR" to a directory containing one of the above files.  If
  "function2" provides a separate development package or SDK, be sure it has
  been installed.


-- Configuring incomplete, errors occurred!
See also "/home/user/test/build/CMakeFiles/CMakeOutput.log".
@Naios
Copy link
Owner

Naios commented Nov 19, 2023

Hi, the documentation also states that it requires function2 and it recommends adding both libraries as git submodule (https://naios.github.io/continuable/installation.html#installation-installation-copy):

git submodule add https://github.com/Naios/continuable.git
git submodule add https://github.com/Naios/function2.git

// ...

add_subdirectory(function2)
add_subdirectory(continuable)

@Spongman
Copy link
Author

Spongman commented Nov 19, 2023

that's not called out in Through CMake. it's not obvious that the instructions listed under "Through CMake" aren't sufficient in themselves.

the documentation could be significantly clearer if it just listed, for each installation method:

  • the commands you need to run
  • the lines you need to add to your CMakeLists.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants