Re: [Freeipa-devel] [PATCH] Debian client support

2014-01-29 Thread Martin Kosek
On 01/27/2014 12:17 PM, Timo Aaltonen wrote: > On 28.11.2013 22:26, Lukas Slebodnik wrote: >> On (05/09/13 23:25), Lukas Slebodnik wrote: >>> On (03/09/13 00:43), Timo Aaltonen wrote: This fixes https://fedorahosted.org/freeipa/ticket/1887 and https://fedorahosted.org/freeipa/ti

Re: [Freeipa-devel] [PATCH] Debian client support

2014-01-27 Thread Timo Aaltonen
On 28.11.2013 22:26, Lukas Slebodnik wrote: > On (05/09/13 23:25), Lukas Slebodnik wrote: >> On (03/09/13 00:43), Timo Aaltonen wrote: >>> >>> This fixes https://fedorahosted.org/freeipa/ticket/1887 >>> and >>> https://fedorahosted.org/freeipa/ticket/2455 >>> >>> the first three patches fix some bu

Re: [Freeipa-devel] [PATCH] Debian client support

2013-11-28 Thread Lukas Slebodnik
On (05/09/13 23:25), Lukas Slebodnik wrote: >On (03/09/13 00:43), Timo Aaltonen wrote: >> >>This fixes https://fedorahosted.org/freeipa/ticket/1887 >>and >>https://fedorahosted.org/freeipa/ticket/2455 >> >>the first three patches fix some bugs in how python is used >>fourth patch checks if dbus is

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-16 Thread Petr Viktorin
On 09/03/2013 11:00 AM, Petr Viktorin wrote: On 09/02/2013 11:43 PM, Timo Aaltonen wrote: This fixes https://fedorahosted.org/freeipa/ticket/1887 and https://fedorahosted.org/freeipa/ticket/2455 Thank you! the first three patches fix some bugs in how python is used These look okay, I'll c

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-05 Thread Lukas Slebodnik
On (03/09/13 00:43), Timo Aaltonen wrote: > >This fixes https://fedorahosted.org/freeipa/ticket/1887 >and >https://fedorahosted.org/freeipa/ticket/2455 > >the first three patches fix some bugs in how python is used >fourth patch checks if dbus is already running before trying to start it >fifth fix

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-03 Thread Timo Aaltonen
On 03.09.2013 12:00, Petr Viktorin wrote: > On 09/02/2013 11:43 PM, Timo Aaltonen wrote: >> >> This fixes https://fedorahosted.org/freeipa/ticket/1887 >> and >> https://fedorahosted.org/freeipa/ticket/2455 > > Thank you! > >> the first three patches fix some bugs in how python is used > > These

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-03 Thread Jakub Hrozek
On Tue, Sep 03, 2013 at 01:34:48PM +0200, Petr Viktorin wrote: > On 09/03/2013 11:22 AM, Jakub Hrozek wrote: > >On Tue, Sep 03, 2013 at 11:00:07AM +0200, Petr Viktorin wrote: > >>>fifth fixes some compilation warnings > >> > >>Looks good to my eyes, perhaps a C expert can look at this one too. > >>

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-03 Thread Petr Viktorin
On 09/03/2013 11:22 AM, Jakub Hrozek wrote: On Tue, Sep 03, 2013 at 11:00:07AM +0200, Petr Viktorin wrote: fifth fixes some compilation warnings Looks good to my eyes, perhaps a C expert can look at this one too. I wonder why these warnings aren't enabled in our builds, though. They look goo

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-03 Thread Jakub Hrozek
On Tue, Sep 03, 2013 at 11:00:07AM +0200, Petr Viktorin wrote: > >fifth fixes some compilation warnings > > Looks good to my eyes, perhaps a C expert can look at this one too. > I wonder why these warnings aren't enabled in our builds, though. They look good to me, too. (Does this answer make me

Re: [Freeipa-devel] [PATCH] Debian client support

2013-09-03 Thread Petr Viktorin
On 09/02/2013 11:43 PM, Timo Aaltonen wrote: This fixes https://fedorahosted.org/freeipa/ticket/1887 and https://fedorahosted.org/freeipa/ticket/2455 Thank you! the first three patches fix some bugs in how python is used These look okay, I'll check when other build errors are fixed. four