On 28.07.2016 18:29, Ariel Barria wrote:
2016-07-28 7:10 GMT-05:00 Petr Spacek <pspa...@redhat.com>:
On 27.7.2016 18:26, Ariel Barria wrote:
2016-07-26 9:39 GMT-05:00 Petr Spacek <pspa...@redhat.com>:
On 26.7.2016 16:28, Jan Cholasta wrote:
On 26.7.2016 16:09, Martin Basti wrote:
On 22.07.2016 00:14, Ariel Barria wrote:
I send patch for review.
NACK, six.u() is supposed to be used on string literals *only* .
A proper fix would be something like:
value = self.to_text()
if not isinstance(value, unicode):
value = value.decode('ascii')
thanks for the guidance and comments
Most importantly, we should fix/provide this method in python-dns and inherit
this method from there.
Well, I made a pr, but apparently travis ci test failed with other
versions of python, so it is possible that they do not approve, I will
be performing other tests to see what the downside.
Looking at the PR, there are functions
What is missing in them?
I will look on this, for some reason we received your e-mail just today
it was my mistake, I sent the patch to the list before being signed to the list
I send the requested changes
thanks for review.
New dnspython always return binary strings, so we should always decode
it, because IPA internals need punycoded domain in unicode format
IMO this is broken in current dnspython released in Fedora. There are
plenty of py3 bugs, we have to provide newer dnspython package to
fedora, so this should not be fixed on IPA side.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code