On 08/18/2014 05:03 PM, thierry bordaz wrote:
On 08/18/2014 04:06 PM, Petr Viktorin wrote:
On 08/14/2014 07:18 PM, thierry bordaz wrote:

    Following Petr remarks from the previous review, I modified the
    original fix to move it only in '.update' files.


Looks better, thanks!
I've tested install and upgrades, everything works as expected.
:-) . Thanks for your tests.

Some whitespace issues:

Applying: User Life Cycle: create containers and scoping DS plugins
.git/rebase-apply/patch:44: new blank line at EOF.
.git/rebase-apply/patch:111: new blank line at EOF.
warning: 2 lines add whitespace errors.

A tiny nitpick:
+dn: cn=Deleted users,cn=accounts,cn=provisioning,$SUFFIX

CN is case-insensitive, but the capital D does stand out.

Good ! This extra lines sounds a familiar issue to me ;). I fixed this
and I removed the extra lines and lowercase.
Basically it creates a new patch

How should I proceed here, send a new review on freeipa-devel and/or
attache this patch (1.3) to the bug ?

Simply reply to this mail with the revised patch attached.

As for attaching patches to the tickets, I've never done it and no one ever complained, so it's not necessary. But attach it if you like :)

Otherwise ACK. We want to push this together with your patch 0002, right?
Well, patch 0002 does not require 0001 (and the opposite as well).
Although they are both related to ULC, I separated them to make reviews
and dev simpler.
So if you are ok, I can push 0001 without the immediate need to push 0002.



Freeipa-devel mailing list

Reply via email to