On Wed, 2011-11-30 at 15:19 +0100, Jan Cholasta wrote: > Dne 23.11.2011 16:46, Martin Kosek napsal(a): > > On Wed, 2011-11-23 at 11:14 +0100, Jan Cholasta wrote: > >> Dne 23.11.2011 09:52, Martin Kosek napsal(a): > > ... > >> > >> Well, the crash is obviously not related to the patch. Fedora 16 uses a > >> newer version of pylint (0.24), which causes the crash (it works fine on > >> Fedora 15 with pylint 0.22). > >> > >> I have opened a ticket to resolve that: > >> https://fedorahosted.org/freeipa/ticket/2136 > > > > The crash _is_ related to the patch since the combination of pylint and > > the changed source code causes it :-) But of course, it does not mean > > that the patch is wrong. > > > > If possible, I would like to push both your acked patch 55 and a patch > > fixing 2136 at one time. I don't want to confuse F-16 users with this > > crash when they try to compile our sources. Therefore, I moved 2136 to > > November milestone. > > > > Martin > > > > Fixed the crash (patch 58, see attachment) and rebased the CSV patch to > current master. > > Honza >
ACK. Both pushed to master. Martin _______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
