pyport.h FreeBSD/Mac OS X "fix" causes errors in C++ compilation

[mbrethen]

I have experienced build errors (example below) with pyport.h The issue is documented in Python's bug tracker here:

​http://bugs.python.org/issue10910

The problem is that pyport tries to replace the ctypes definion of the isascii(ch) and related functions by a replacement that works better with UTF-8 on FreeBSD and OSX. That replacement is incompatible with the localfwd.h header.

A workaround is provided at the site, which I'm attaching.

:info:build /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../lib/c++/v1/__locale:436:15: error: C++ requires a type specifier for all declarations
:info:build     char_type toupper(char_type __c) const
:info:build               ^~~~~~~~~~~~~~~~~~~~~~
:info:build /opt/local/Library/Frameworks/Python.framework/Versions/2.7/Headers/pyport.h:731:29: note: expanded from macro 'toupper'

:info:build /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../lib/c++/v1/__locale:442:48: error: too many arguments provided to function-like macro invocation
:info:build     const char_type* toupper(char_type* __low, const char_type* __high) const
:info:build                                                ^
:info:build /opt/local/Library/Frameworks/Python.framework/Versions/2.7/Headers/pyport.h:731:9: note: macro 'toupper' defined here
:info:build #define toupper(c) towupper(btowc(c))
:info:build         ^

[mf2k (Frank Schima)]

In the future, please fill in the Port field and actually Cc the port maintainers by running the following command in Terminal to get them - (port info --maintainers python27).

[mbrethen]

I patched my local python installation and confirmed the issue resolved. I have uploaded patches for pyport.h and the Portfile.

[jyrkiwahlstedt]

done in r122030, thanks

[mamoll (Mark Moll)]

I think python34 and python35 need the same kind of patch.

[kylegoetz@…]

Is there anyone who can implement a similar fix for python34? I am having problems installing a package, and the package maintainer linked me to this and said this was the issue.

[mbrethen]

Have you tried patching it locally to confirm it works? See ​https://trac.macports.org/wiki/howto/PatchLocal

[mamoll (Mark Moll)]

Replying to kylegoetz@…:

Is there anyone who can implement a similar fix for python34? I am having problems installing a package, and the package maintainer linked me to this and said this was the issue.

You can manually apply the change in the patch-Include-pyport.h.diff​ attachment to /opt/local/Library/Frameworks/Python.framework/Versions/3.4/include/python3.4m/pyport.h and then try port install py34-graph-tool again.

[mamoll (Mark Moll)]

Replying to mark.brethen@…:

Have you tried patching it locally to confirm it works? See ​https://trac.macports.org/wiki/howto/PatchLocal

I can confirm that the patch works for python35.

[mbrethen]

I suggest opening a new ticket for python3 with the patch and updated portfile.