[SSSD] [PATCH] Display more information on DB version crash

Ondrej Kos okos at redhat.com
Wed Nov 14 15:20:52 UTC 2012


On 11/14/2012 03:38 PM, Simo Sorce wrote:
> On Wed, 2012-11-14 at 15:18 +0100, Jan Cholasta wrote:
>>
>> Just one more nitpick: SSS_DB_CHECK_PTS and sss_db_version_check are
>> used only in sysdb.c, so there is no reason to have them defined
>> publicly in util.h+util.c. Move them both to sysdb.c please (you
>> might
>> also want to rename them to better fit in sysdb.c).
>>
>
> Yes please, rename them to be sysdb_* functions and move them in sysdb
> where they belong.
> The util/ shouldn't have component specific functions.
>
> Simo.
>

moved all code to sysdb, new patch attached.

O.


-- 
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech

phone: +420-532-294-558
cell:  +420-736-417-909
ext:   82-62558
loc:   1013 Brno 1 office
irc:   okos @ #brno
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Display-more-information-on-DB-version-crash.patch
Type: text/x-patch
Size: 7801 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20121114/313edf5b/attachment.bin>


More information about the sssd-devel mailing list