On 08/23/2016 10:58 AM, Petr Cech wrote:
> Hello,
>
> there is new patch number 3 which is a WIP.
> I would like to ask you which version
> of ldif printing do you prefer.
>
> You can see [1] which shows the outputs.
>
> Difference is that version A is step by step
> reading of ldb messsage.
>
> But, version B uses function
> ldb_ldif_write(). This function needs
> my_vprintf_fn() but the information isn't structured.
>
> What do you prefer? :-)
>
> [1] ldif.txt
There is new patch set attached.
Well, I choose version B because it is uses native ldb function
ldb_ldif_write().
I added new debug level. It is not only extra debug level which SSSD
has. But I didn't see any other extra in man pages. So I didn't add
this one too.
Bump.
Notes:
1) If you prefer merging of last 4 patches -- renaming of ldb_*(),
please, tell me.
2) Those patches rename only ldb_*() in sysdb. But those functions are
in other parts of code too. So if you prefer renaming in whole SSSD,
please tell me.
Regards
--
Petr^4 Čech