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

Not found with conan + cmake_paths #538

Open
0Xellos opened this issue Aug 18, 2021 · 4 comments
Open

Not found with conan + cmake_paths #538

0Xellos opened this issue Aug 18, 2021 · 4 comments

Comments

@0Xellos
Copy link

0Xellos commented Aug 18, 2021

CMake gives the standard "package not found" error

  By not providing "Findxsimd.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "xsimd", but
  CMake did not find one.

  Could not find a package configuration file provided by "xsimd" with any of
  the following names:

    xsimdConfig.cmake
    xsimd-config.cmake

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

See minimum failing example: test.zip.

To make it work, we're using the following patches in a custom conan package: patch.zip. I'm not sure if that's the "right" way to fix and it's only a sidenote in other work, but if we could use xsimd from conancenter directly that'd be good long-term.

@SylvainCorlay
Copy link
Member

Which version of xsimd are you using?

@0Xellos
Copy link
Author

0Xellos commented Aug 18, 2021

We're using 7.4.0 in our patched builds, but it isn't in conancenter now so the example has 7.5.0.

@SylvainCorlay
Copy link
Member

Thanks, I just wanted to make sure that you are not using master.

This is surprising because xsimd does provide a xsimdConfig.cmake . However, we (xtensor and xsimd maintainers) don't maintain the conan packages, but only the conda ones, and it is possible that something is wrong with how this has been packaged for conan.

@0Xellos
Copy link
Author

0Xellos commented Aug 19, 2021

Yeah, that should be the case. Perhaps specifically for the cmake_paths generator.

Where's the right place to put the issue then?

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