Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-08-01 Thread Petr Spacek
On 1.8.2016 18:31, Martin Basti wrote: > > > On 28.07.2016 18:29, Ariel Barria wrote: >> 2016-07-28 7:10 GMT-05:00 Petr Spacek : >>> On 27.7.2016 18:26, Ariel Barria wrote: 2016-07-26 9:39 GMT-05:00 Petr Spacek : > On 26.7.2016 16:28, Jan Cholasta wrote: >> Hi, >> >> On 26.7.

Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-08-01 Thread Martin Basti
On 28.07.2016 18:29, Ariel Barria wrote: 2016-07-28 7:10 GMT-05:00 Petr Spacek : On 27.7.2016 18:26, Ariel Barria wrote: 2016-07-26 9:39 GMT-05:00 Petr Spacek : On 26.7.2016 16:28, Jan Cholasta wrote: Hi, On 26.7.2016 16:09, Martin Basti wrote: On 22.07.2016 00:14, Ariel Barria wrote: H

Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-07-28 Thread Ariel Barria
2016-07-28 7:10 GMT-05:00 Petr Spacek : > On 27.7.2016 18:26, Ariel Barria wrote: >> 2016-07-26 9:39 GMT-05:00 Petr Spacek : >>> On 26.7.2016 16:28, Jan Cholasta wrote: Hi, On 26.7.2016 16:09, Martin Basti wrote: > > > On 22.07.2016 00:14, Ariel Barria wrote: >> Hello

Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-07-28 Thread Petr Spacek
On 27.7.2016 18:26, Ariel Barria wrote: > 2016-07-26 9:39 GMT-05:00 Petr Spacek : >> On 26.7.2016 16:28, Jan Cholasta wrote: >>> Hi, >>> >>> On 26.7.2016 16:09, Martin Basti wrote: On 22.07.2016 00:14, Ariel Barria wrote: > Hello everyone. > > I send patch for review. >>>

Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-07-26 Thread Petr Spacek
On 26.7.2016 16:28, Jan Cholasta wrote: > Hi, > > On 26.7.2016 16:09, Martin Basti wrote: >> >> >> On 22.07.2016 00:14, Ariel Barria wrote: >>> Hello everyone. >>> >>> I send patch for review. > > NACK, six.u() is supposed to be used on string literals *only* [1]. > > A proper fix would be somet

Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-07-26 Thread Jan Cholasta
Hi, On 26.7.2016 16:09, Martin Basti wrote: On 22.07.2016 00:14, Ariel Barria wrote: Hello everyone. I send patch for review. NACK, six.u() is supposed to be used on string literals *only* [1]. A proper fix would be something like: value = self.to_text() if not isinstance(value,

Re: [Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-07-26 Thread Martin Basti
On 22.07.2016 00:14, Ariel Barria wrote: Hello everyone. I send patch for review. Regards, Thank you, I will look on this, for some reason we received your e-mail just today (2016-07-26) Martin -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mai

[Freeipa-devel] [PATCH] 0001 six.u function instead of the decode

2016-07-26 Thread Ariel Barria
Hello everyone. I send patch for review. Regards, From 7aaf440617963facf7bd156f1e7295562391a3f2 Mon Sep 17 00:00:00 2001 From: "Ariel O. Barria" Date: Thu, 21 Jul 2016 17:06:05 -0500 Subject: [PATCH] freeipa arielb 0001 six.u function instead of the decode function to avoid DNSName.ToASCII bro