On (18/01/16 18:03), Simo Sorce wrote:
>Found this while working on another patch.
>
>It is not evident by this patch alone but ... "trust me" :-)
>(I'll send the other patch next, try to apply just that one and see what
>I mean if you want)
>
Then probably you have something bad in your patches.

It's best practice to include system beader file before
local header file and you changed that.

I will wait for your another patch and I will try to propose
different solution :-)

tennative NACK

LS
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to