conda-forge / sundials-feedstock

A conda-smithy repository for sundials.
BSD 3-Clause "New" or "Revised" License
4 stars 10 forks source link

Build static win32 libs #2

Closed jschueller closed 8 years ago

jschueller commented 8 years ago

sundials_kinsol.dll doesn't export all of it's symbols when building assimulo package https://github.com/conda-forge/staged-recipes/pull/1727:

error LNK2019: unresolved external symbol _KINProcessError referenced in function _KINPinvSetJacFn

This switches from shared libs to static on win target (I tried providing both but the import and static libs names clash).

conda-forge-linter commented 8 years 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) and found it was in an excellent condition.

bjodah commented 8 years ago

What do you mean by "import and static libs names clash"? Does the python extension module have the same name as the sundials provided shared library?

jschueller commented 8 years ago

The import lib and static lib of sundial.kinsol have the same name: sundial_kinsol.lib. As the shared lib is installed last it overwrites the static one.

bjodah commented 8 years ago

So it's an upstream problem in the CMake files in sundials really? Any chance of patching those?

bjodah commented 8 years ago

Sorry, I'm a bit lost in windows land, but aren't static libraries in windows named .lib and dynamic ones .dll? If so, how could there be a name clash?

jschueller commented 8 years ago
bjodah commented 8 years ago

Thank you for the explanation. I'm fine with using static libraries on windows if that simplifies things.

bjodah commented 8 years ago

Pinging @patricksnape who helped me get the recipe building on windows in the first place.

bjodah commented 8 years ago

Merging this, thanks @jschueller