Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-19 Thread Jan Cholasta
On 15.1.2016 13:43, Tomas Babej wrote: On 01/12/2016 10:24 AM, Jan Cholasta wrote: On 6.1.2016 12:33, Christian Heimes wrote: On 2016-01-05 11:30, Tomas Babej wrote: On 01/05/2016 08:54 AM, Jan Cholasta wrote: Hi, the attached patch replaces the default_encoding_utf8 binary module with 2

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-15 Thread Tomas Babej
On 01/12/2016 10:24 AM, Jan Cholasta wrote: > On 6.1.2016 12:33, Christian Heimes wrote: >> On 2016-01-05 11:30, Tomas Babej wrote: >>> >>> >>> On 01/05/2016 08:54 AM, Jan Cholasta wrote: Hi, the attached patch replaces the default_encoding_utf8 binary module with 2 lines

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-12 Thread Jan Cholasta
On 6.1.2016 12:33, Christian Heimes wrote: On 2016-01-05 11:30, Tomas Babej wrote: On 01/05/2016 08:54 AM, Jan Cholasta wrote: Hi, the attached patch replaces the default_encoding_utf8 binary module with 2 lines of equivalent Python code. Honza This looks fine to me, however, I wonder,

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-06 Thread Christian Heimes
On 2016-01-05 11:30, Tomas Babej wrote: > > > On 01/05/2016 08:54 AM, Jan Cholasta wrote: >> Hi, >> >> the attached patch replaces the default_encoding_utf8 binary module with >> 2 lines of equivalent Python code. >> >> Honza >> >> >> > > This looks fine to me, however, I wonder, why this approa

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-05 Thread Alexander Bokovoy
On Tue, 05 Jan 2016, Jan Cholasta wrote: On 5.1.2016 11:54, Alexander Bokovoy wrote: On Tue, 05 Jan 2016, Tomas Babej wrote: On 01/05/2016 08:54 AM, Jan Cholasta wrote: Hi, the attached patch replaces the default_encoding_utf8 binary module with 2 lines of equivalent Python code. Honza

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-05 Thread Jan Cholasta
On 5.1.2016 11:54, Alexander Bokovoy wrote: On Tue, 05 Jan 2016, Tomas Babej wrote: On 01/05/2016 08:54 AM, Jan Cholasta wrote: Hi, the attached patch replaces the default_encoding_utf8 binary module with 2 lines of equivalent Python code. Honza This looks fine to me, however, I wonder

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-05 Thread Alexander Bokovoy
On Tue, 05 Jan 2016, Tomas Babej wrote: On 01/05/2016 08:54 AM, Jan Cholasta wrote: Hi, the attached patch replaces the default_encoding_utf8 binary module with 2 lines of equivalent Python code. Honza This looks fine to me, however, I wonder, why this approach was ever taken? The sys.s

Re: [Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-05 Thread Tomas Babej
On 01/05/2016 08:54 AM, Jan Cholasta wrote: > Hi, > > the attached patch replaces the default_encoding_utf8 binary module with > 2 lines of equivalent Python code. > > Honza > > > This looks fine to me, however, I wonder, why this approach was ever taken? The sys.setdefaultencoding is avail

[Freeipa-devel] [PATCH 536] ipapython: remove default_encoding_utf8

2016-01-04 Thread Jan Cholasta
Hi, the attached patch replaces the default_encoding_utf8 binary module with 2 lines of equivalent Python code. Honza -- Jan Cholasta From c69fece2fb2bcc68c7efec841178ea4cb4cfd48d Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Tue, 5 Jan 2016 08:49:04 +0100 Subject: [PATCH] ipapython: rem