-
Notifications
You must be signed in to change notification settings - Fork 201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Building on Windows does not create shared Boost.Numpy #98
Comments
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 |
Any help on this would be greatly appreciated. We're kind of at a loss ATM. |
It is the other way round. Building the shared libraries fails as the missing |
Yeah, sorry, we just realized we had it backwards earlier. So what sort of change would we be looking at? |
Hi ! I stumbled on that issue yesterday trying to get rid of the old (independent) boost numpy. |
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. |
I could make an effort on fixing this issue if nobody else has free time to devote to resolving this... |
Hi It would be a good idea. I don't use much windows and i only port my work from linux. |
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. |
I have just posted a PR at #108 in an attempt to fix this. I'd appreciate if someone could confirm, before I merge. |
Fix for missing export symbols in shared library of boost::python::numpy
So I guess this should be fixed by PR ( #110 ). Will try to test this and let you know how it goes. |
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. |
It seems to work correctly! |
Thank you very much !
2017-02-16 7:17 GMT+01:00 Michele Dolfi <notifications@github.com>:
… 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>
.
|
Missed linking of boost numpy submodule
This is due to a bug in 1.63 which means the boost_numpy.lib is not created. boostorg/python#98
This is a necessary prerequisite to #70. Reason this failed was that BOOST_PYTHON_SOURCE was defined for the boost::numpy components. This led to an attempt to export boost::python in the boost::numpy library, which then failed during linking. On Windows boost::numpy requires boost 1.64. This is due to a bug in 1.63 which means the boost_numpy.lib is not created. boostorg/python#98
This is a necessary prerequisite to #70. Reason this failed was that BOOST_PYTHON_SOURCE was defined for the boost::numpy components. This led to an attempt to export boost::python in the boost::numpy library, which then failed during linking. On Windows boost::numpy requires boost 1.64. This is due to a bug in 1.63 which means the boost_numpy.lib is not created. boostorg/python#98
This is a necessary prerequisite to #70. Reason this failed was that BOOST_PYTHON_SOURCE was defined for the boost::numpy components. This led to an attempt to export boost::python in the boost::numpy library, which then failed during linking. On Windows boost::numpy requires boost 1.64. This is due to a bug in 1.63 which means the boost_numpy.lib is not created. boostorg/python#98
At conda-forge/boost-feedstock#32 we are trying to finalize the Boost Python package on Conda.
The build currently fails while copying
boost_numpy-vc90-mt-1_63.lib
, because the file does not exist.The full log is at: boost_numpy-vc90-mt-1_63.lib
The build command is:
Is there something wrong in the build process, or did we find an issue in the Boost building process?
The text was updated successfully, but these errors were encountered: