CMake config is not included

[ianhbell]

I am building a nanobind interface for nlohmann::json, the nanobind variant of pybind11_json. I cannot use all their tooling directly because the CMake config is not a part of the nanobind conda package. For pybind11 we have the necessary file here:

/Users/ihb/mambaforge/envs/py312/lib/python3.12/site-packages/pybind11/share
├── cmake
│   └── pybind11
│       ├── FindPythonLibsNew.cmake
│       ├── pybind11Common.cmake
│       ├── pybind11Config.cmake
│       ├── pybind11ConfigVersion.cmake
│       ├── pybind11GuessPythonExtSuffix.cmake
│       ├── pybind11NewTools.cmake
│       ├── pybind11Targets.cmake
│       └── pybind11Tools.cmake
└── pkgconfig
    └── pybind11.pc

but the same nanobindConfig.cmake file does not exist for nanobind. Can you please add it?

[ianhbell]

There is a nanobind-config.cmake:

/Users/ihb/mambaforge/envs/py312/lib/python3.12/site-packages/nanobind/cmake
├── darwin-ld-cpython.sym
├── darwin-ld-pypy.sym
├── nanobind-config-version.cmake
└── nanobind-config.cmake

but CMake does not find it, even when this package is installed via conda

[ianhbell]

I see - is that documented somewhere? scikit-build-core does not get invoked when building the tests

[ianhbell]

It would be convenient if no magical incantations would be required to get nanobind stuff pulled in to minimize migration headaches

[wjakob]

I don't work with Conda myself and just do the minimal version bumps to keep the package in sync. Note that the python -m nanobind --cmake_dir incantation is actually the official way to get nanobind's CMake include directory, and this is also documented. But it could be the case that it only includes nanobind-config.cmake while lacking some auto-generated CMake files. If you have suggestions and improvements, it would be great if you can open a PR on the feedstock.

[ianhbell]

Where is the feedstock? I think it is just a question of folder structure if I understand correctly. I can see about fixing it.

[wjakob]

in the conda-forge master repository like all of the other feedstocks.

[RaulPPelaez]

For the record, CMake's FetchContent seems to work fine:

include(FetchContent)
FetchContent_Declare(
  nanobind
  GIT_REPOSITORY https://github.com/wjakob/nanobind/
  GIT_TAG        v2.4.0
)
FetchContent_MakeAvailable(nanobind)