Support Windows?

[DirectXMan12]

We may wish to consider supporting Windows at some point

[filmor]

What needs to be done to get there?

[DirectXMan12]

1. Compilation needs to be tested in Windows with MIT Kerberos for Windows on MinGW.
2. The conditional compilation logic needs to be adjusted to work for Windows
3. We need a way to build wheel files for Windows (optional, but useful for anyone who actually wants to use this on Windows).

[frozencemetery]

Other "nice to have" - some way to run our test suite, since I don't think k5test will work there.

[ibigpapa]

So using the MIT KfW. I see the source is available as well as precompiled sources could we just wrap these to make it work?

[DirectXMan12]

yeah, it really comes down to finding time and a Windows box to test on (plus setting up CI to make sure we don't break anything), plus tweaking setup.py to work properly on Windows.

[ibigpapa]

I'm working on trying to hack this in myself let you know how i do.

[DirectXMan12]

cool. Feel free to ping us on #python-gssapi on freenode if you have any questions best answered in a real-time manner (I'm usually on from 10:00-17:00EST)

[frozencemetery]

2a2b47b adds support for conditional compilation. Still missing are building wheels and running buildjob CI. Much like with Heimdal, a stretch goal is to port k5test so we can run the test suite there.

[aiudirog]

I've been working on Windows support the past couple of days to get my company to move off of requests-kerberos (I had previously patched ccs-kerberos to work on Windows) and got everything to compile properly under VS Build Tools, MinGW, and MSYS. Unfortunately I won't be able to test until I'm back at work on Monday since k5test isn't ported yet. My current progress can be found here.

Changes:

• I created a predefined macro (__MSYS__) to allow the header files to detect MinGW under MSYS so that when it isn't under it, they can target MIT Kerberos for Windows instead (which also seems to have better extension support).
• To get it compile under VS:
  • I had to cast all slicing of gss_buffer_desc.values to char* because many platforms seem to declare it as a void* instead and cl.exe doesn't like void* pointer arithmetic. This can be found a separate commit.
  • libraries is now passed separately to Extension like library_dirs so that Extension itself can determine how to pass them to the linker.
• link_args and compile_args are now always lists. This is to support arguments which may have spaces in them, like include paths on Windows.

[aiudirog]

Everything seems to be working as expected. I was able to connect to our internal Apache Livy and Phoenix servers on Windows and use them as normal through requests-gssapi.

[frozencemetery]

That's awesome, well done! Do you have any idea how this could be tested in CI? (Presumably just checking that the build works, as we do with Heimdal, since the krb5 server parts don't run on Windows, but don't let me stop you if you're feeling ambitious.)

[aiudirog]

I've been looking into it and Travis does support Windows now so I'll put some work into at the very least getting it to build and generate wheels.