[Freeipa-devel] [PATCH] 55 Parse comma-separated lists of values in all parameter types

[Jan Cholasta]

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

-- 
Jan Cholasta
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-55.3-comma-separated-list.patch
Type: text/x-patch
Size: 131989 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20111130/d7baa186/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-58-make-lint-crash.patch
Type: text/x-patch
Size: 773 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20111130/d7baa186/attachment-0001.bin>