Fix Python-related redefinitions for builds using mingw-w64#141
Fix Python-related redefinitions for builds using mingw-w64#141sav-ix wants to merge 1 commit intoboostorg:developfrom sav-ix:develop
Conversation
|
By "broke MSVC builds", are you referring to the Appveyor build failure ? That's an unrelated issue with the SCons build logic. Boost.Build is not involved in the CI builds. If not, can you show an error log ? (I'm only a casual Windows user, so I may not be able to help.) |
|
Travis failures forced me to test this patch for MSVC builds. Which I missed to do before publishing this PR. (see logs in attachment). |
| project boost/python | ||
| : source-location ../src | ||
| : requirements <toolset>gcc <target-os>windows:<cxxflags>-D_hypot=hypot | ||
| : requirements <toolset>gcc <target-os>windows <link>shared:<cxxflags>-DMS_WIN64 |
There was a problem hiding this comment.
I don't think this is doing what you expect. As far as I understand Boost.Build syntax, the last line above defines as requireents:
- toolset=gcc
- target-os=windows
- cxxflags=-DMS_WIN64, whenever link=shared
It looks like what you intend to do is to define -DMS_WIN64 whenever all three features (toolset, target-os, and link) have the given values, yes ?
Please ask on the boost or boost.build mailing list what the right syntax for this is.
(I'm also not sure how this would fix the `"gcc is not a known value..." error. That's another thing to ask on boost.build ML).
Sorry I can't help more with this. Boost.Build is complicated and badly documented, which is one reason why I'm trying to move away from it...
|
I was trying to add rules to Boost.Python build system:
Or we can invite Boost.Build Developers to join this discussion. |
Fixes #140.
Tested for builds using mingw-w64 with
<Debug|Release><Shared|Static>configurations.UPDATE: But it broke MSVC builds. Any suggestions?