On Mon, Apr 02, 2012 at 10:07:00AM +0200, Jan Zeleny wrote:
Ok, all those changes took me a while but I'm finally done.
Please note that
patch #0003 has been split into two and the part with utility functions has
been moved right after sysdb routines. There is a circular dependency between
the two which I don't think is worth trying to solve, neither patch would work
without the other. This patch split caused the numbering to be shifted by one.
Yes, patch #1 uses new_subdomain() in tests, patch #2 uses
sss_domain_info->conn_name.
In general, I think each patch should compile on its own for two
reasons:
1) to ease cherry-picking
2) to allow git-bisect
No 1 is a moot point with a patch set like this because we would be
cherry-picking the entire patch set anyway. However, I use git-bisect
from time to time, and I'd like to avoid breaking it if possible.