On Thu, Sep 11, 2014 at 02:47:33PM +0200, Šrubař Michal wrote:
Cituji Pavel Březina <pbrezina(a)redhat.com>:
>On 09/08/2014 03:47 PM, Jakub Hrozek wrote:
>>On Wed, Aug 20, 2014 at 01:40:45PM +0200, Šrubař Michal wrote:
>>>Hi guys, I spent some time working at this ticket
>>>https://fedorahosted.org/sssd/ticket/1108 and I think it's finally ready
to
>>>be reviewed by others.
>>>
>>>Description of the problem and scope of the changes can be found in the
>>>commit message. I also wrote some unit tests but the patch is a quite long
>>>already so I think it would be better to send the tests as an another
>>>patch.
>>>Or should I create a patch for each modified file?
>>>
>>>
>>
>>Thank you very much for taking the time to finish the code after your
>>thesis was done!
>>
>>I'm bound to other tasks at the moment, though, I wonder if Pavel could
>>find some time to review the patch?
>
>I'm going to PTO so unfortunately no. However from the first very quick
>look I think the concept is OK. It would be nice to have the changes made
>in more commits instead of one huge, but well... what can you do now.
I can break up the patch into smaller pieces so it can be more easily
reviewed. But I am also a bit busy at the moment but I will do it asap.
No hurry if you're busy and thanks for keeping in touch. But I agree
with Pavel that several patches would be easier to digest.