PATCH: python3 smb module
Andrew Bartlett
abartlet at samba.org
Tue Feb 13 21:44:46 UTC 2018
On Tue, 2018-02-13 at 12:43 +0000, Noel Power via samba-technical
wrote:
> I noticed I erroneously specified the smb_methods in the module def,
> this doesn't reflect how the previous python 2 code was using the
> Py_InitModule3 code. I've changed the patch accordingly
>
> On a sidenote I wonder should we change the api here for the smb module.
> I have noticed that loadfile (which although documented to load
> filecontents as a string) doesn't seem to handle content starting with
> unicode BOM :/ perhaps returning a byte string might be a better choice
> (for the python3 variant)
> I'm working on a test to demonstrate this but python2/python3 is fighting me
Aside from that (which I'll try and understand later) please ensure the
patch passes 'make test TESTS=source'.
Thanks!
Andrew Bartlett
--
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team https://samba.org
Samba Development and Support, Catalyst IT
https://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list