[PATCH] samba.registry Python 3 compatible and Python 2.6 fixes
Andreas Schneider
asn at samba.org
Thu Oct 12 13:04:58 UTC 2017
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
--
Andreas Schneider GPG-ID: CC014E3D
Samba Team asn at samba.org
www.samba.org
More information about the samba-technical
mailing list