update cmake files for conda build

[Olllom]

Hi @beniz,

in order to make the code build on conda, we had to patch up the CMake configuration a bit.

Whether the python libraries have to be linked depends on the operating system:

• On Windows, they need to be linked
• On MacOS, the Python module needs to be linked with -undefined dynamic_lookup
• On Linux, they don't need to be linked

This PR takes care of this logic so that we don't need the LINK_PYTHON option any longer.

Furthermore, the CMake build location for the library is ${PROJECT_BINARY_DIR}\\src by default on Windows, which is added to the PATH for the C tests.

Tagging @jschueller

Best,
Andreas

[beniz]

Hi, let me know when you need this merged, and if you need it despite the CI failure.

[jschueller]

I guess there's still an issue with the dll path for tests, maybe retry without the path change ?

[Olllom]

OK, ready to merge. Putting the more specific path first passed the CI.

[beniz]

Done. Do you need yet another release ?

[Olllom]

Cool. Maybe not quite yet. Windows is somehow very picky about the order of the PATH directories. Therefore, we would still need a patch in the feedstock even with the current master. Let me try a little bit if I get a version to pass the tests on both your CI and the conda-forge CI. If this succeeds, I would be all for releasing a version that builds on conda-forge without patches.