[PATCH] samba.registry Python 3 compatible and Python 2.6 fixes

[Lumir Balhar]

I found out that I need to add one more #if to supress warning about 
unused PamTestError__doc__ in Python 2.6.

Sorry for the complication. I'll test it again in all Pythons and then 
I'll send new patches for samba and also for pam_wrapper.

Lumír


On 10/12/2017 03:04 PM, Andreas Schneider wrote:
> On Thursday, 12 October 2017 11:30:28 CEST Lumir Balhar via samba-technical
> wrote:
>> Hello.
>>
>> I ported another Python module, samba.registry, to Python 3 compatible
>> form (patch #1) and I improved its test coverage (patch #2).
>>
>> Everything went well with Python 2.7 and 3.6 but then I found some
>> issues during testing the ported module with Python 2.6.6 in VM with
>> fresh CentOS 6 installation.
>> The second two patches contain fixes for Python 2.6.
>>
>> The first problem I found is in lib/pam_wrapper/python/pypamtest.c where
>> function PyErr_NewExceptionWithDoc is used, but this function is
>> available only in Python 2.7 and higher. This bug was probably hidden
>> because pam_wrapper can be installed as a distribution package. Fixed
>> with #if/#else in patch 3.
> Please also create a patch against:
> https://git.samba.org/?p=pam_wrapper.git;a=summary
>   
>> The second one is in testsuite/unittests/rpc_test_dummy_module.c where I
>> get an error about undefined kill() function and SIGKILL constant. I
>> don't know where this bug comes from, but including signal.h header file
>> fixes it. Fix attached as patch 4.
> RB+
>
> Second team member needed for review.
>
>
> Cheers,
>
>
> 	Andreas
>
>