conda-forge / scipoptsuite-feedstock

A conda-smithy repository for scipoptsuite.
BSD 3-Clause "New" or "Revised" License
25 stars 13 forks source link

Debug cmake-issues triggered by SCIP 9.2.0 #79

Closed discopt closed 4 hours ago

discopt commented 1 week ago

This is not intended for merging but for debugging cmake issues.

conda-forge-admin commented 1 week ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition. I do have some suggestions for making it better though...

For recipe/meta.yaml:

Documentation on acceptable licenses can be found here.

scip-ci commented 1 week ago

@DominikKamp says that he can reproduce this issue on his machine, using a cmake build of the scip optsuite.

@DominikKamp, maybe you can run cmake with --trace and attach the output.

DominikKamp commented 1 week ago

I can avoid the error when I dump

set(CMAKE_MODULE_PATH "/usr/local/lib/cmake/papilo")

in front of

find_dependency(GMP)

in papilo-config.cmake since otherwise the variable is just empty.

cmake.log

discopt commented 1 week ago

I can avoid the error when I dump

set(CMAKE_MODULE_PATH "/usr/local/lib/cmake/papilo")

in front of

find_dependency(GMP)

in papilo-config.cmake since otherwise the variable is just empty.

But this will probably not fix it under Windows because that directory is just not there! I created a PAPILO-internal issue and a merge request to address this properly.

DominikKamp commented 1 week ago

Of course, this should only indicate that papilo-config.cmake is not aware of the location of the corresponding cmake module (although it already lives there).

scip-ci commented 4 hours ago

77 has a working solution, using a patch authored by @discopt.