search

ariovistus / pyd

Interoperability between Python and D
MIT License
158 stars 32 forks source link

Fix #67 - support for 64-bit Python on Windows #72

Closed atilaneves closed 6 years ago

atilaneves commented 6 years ago

PyD on Windows currently assumes that dmd is being used with -m32, which is the default. However, if 64-bit Python is installed, that obviously won't work. This PR adds -m64 to the compiler options if 64-bit Python is being used, and restores the Python library dirs that should be passed to the linker that get monkey patched out of distutils.

thewilsonator commented 6 years ago

Much nicer. I was going to put an appveyor.yml in my pr so that Win64 is tested, but if your pr is going to be used, then it should go with yours. Not sure if @ariovistus has registered with Appveyor yet.

ariovistus commented 6 years ago

https://ci.appveyor.com/project/ariovistus/pyd

atilaneves commented 6 years ago

@thewilsonator I just inspected to see if there was an appveyor build already. Didn't find one and moved on.