Building on Windows does not create shared Boost.Numpy about python

[SPKorhonen]

I could make an effort on fixing this issue if nobody else has free time to devote to resolving this...

[jakirkham]

I think the link to the log got lost. Linked below.

Note: This seems to happen with all Boost.Python builds on Windows, but not on Unix.

ref: https://ci.appveyor.com/project/conda-forge/boost-feedstock/build/1.0.97

[jakirkham]

Any help on this would be greatly appreciated. We're kind of at a loss ATM.

[jhunold]

It is the other way round. Building the shared libraries fails as the missing lib-prefix indicates the shared library. This is no wonder as the necessary dllexport/-import magic is completely missing from Boost.Numpy. The msvc linker thus refuses to create the empty boost_numpy-vc90-mt-1_63.lib stub which is needed by the installroutine.

[jakirkham]

Yeah, sorry, we just realized we had it backwards earlier.

So what sort of change would we be looking at?

[XWarin]

Hi !

I stumbled on that issue yesterday trying to get rid of the old (independent) boost numpy.
Is this there something scheduled to correct this dynamic link problem ?
Best regards

[SPKorhonen]

I also stumbled onto this problem when trying to update boost to 1.63 to get rid of separate boost::numpy dependency.

The problem is that, at least on windows, numpy generates empty shared libraries (the dll is generated but exports no symbols). This is due to the windows dll default settings. Everything that is not explicitly exported is private. As @jhunold stated the numpy is missing dllexport/-import magic words. I think that liberal sprinkling of BOOST_PYTHON_DECL before all symbols that are to be exported would help.

[XWarin]

Hi

It would be a good idea. I don't use much windows and i only port my work from linux.
Right now when on windows i still recompile the old boost numpy.
Best regards

[jakirkham]

Seconding @XWarin. Some of us have limited knowledge of building on Windows and/or time to devote to this. As it sounds like you have both knowledge and time, @SPKorhonen, I'd say go for it. We would be very appreciative. Can help test patches if you would like.

[stefanseefeld]

I have just posted a PR at #108 in an attempt to fix this. I'd appreciate if someone could confirm, before I merge.

[jakirkham]

So I guess this should be fixed by PR ( #110 ). Will try to test this and let you know how it goes.

[dolfim]

We added the patch to the conda-forge package. Now it will try building here: https://ci.appveyor.com/project/conda-forge/boost-feedstock/build/1.0.109

Thanks a lot for the patch.

[dolfim]

It seems to work correctly!

[XWarin]

Thank you very much ! 2017-02-16 7:17 GMT+01:00 Michele Dolfi <[email protected]>:

…

Closed #98 <#98>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#98 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ALThAaWS9w0hWP6luMqW0Ktre1WMlFlaks5rc-oJgaJpZM4Lra8B> .