[PATCH] Porting Samba's CPython extensions to Python 3

Petr Viktorin pviktori at redhat.com
Thu Jul 28 18:21:33 UTC 2016


On 07/14/2016 11:56 AM, Petr Viktorin wrote:
> On 07/11/2016 05:21 PM, Stefan Metzmacher wrote:
>> Hi Petr,
>>
>>> Sorry for the delay – I've been working on porting FreeIPA for a few
>>> months.
>>>
>>> I've spoken with Alexander Bokovoy earlier, and we agreed that adding a
>>> third_party module might be too much overhead for what is essentially a
>>> single header file.
>>>
>>> So, I present two options, and I'm attaching patches that port the
>>> "samba.credentials" module as example usage.
>>> - third_party/py3c, a copy of the (MIT-licensed) py3c project, with
>>> corresponding build system integration. (I'm not sure if I did the
>>> integration correctly; there's probably a better way to add a
>>> header-only third-party library to Waf-samba)
>>> - lib/py3compat.h, a relatively simple header with the necessary macros.
>>> (I forked this from py3c when I was still the sole author, but the
>>> latter additions are not relevant to Samba.)
>>
>> I'd prefer a single py3compat.h it's simple enough, but I'd located it
>> under
>> python/ or python/compat/ instead of lib/
>>
>> metze
>
>
> That does sound better. Here is a patchset that puts the header in python/.

Hello,
Is there anything I can do to help this patch get reviewed?


-- 
Petr Viktorin



More information about the samba-technical mailing list