[SSSD] [PATCH] DB: Switch to new libini_config API

Ondrej Kos okos at redhat.com
Wed Apr 10 14:17:42 UTC 2013


On 03/29/2013 01:19 PM, Jakub Hrozek wrote:
> On Thu, Mar 28, 2013 at 03:59:25PM +0100, Ondrej Kos wrote:
>> Hi,
>>
>> Attached is patch for https://fedorahosted.org/sssd/ticket/1786
>>
>> Patch extends sssd code so it's capable to build with current
>> version of libini_config (0.7 at the time, supported versions up
>> from 0.6.1) and with version 1.0.0, which will be released soon.
>>
>
> I can use this patch as an interim solution in rawhide and F19 until we
> ack it but it needs some work.
>
> Mainly, please don't use ifdefs in general code. ifdefs should be used in
> a wrapper module to provide kind of generic methods the user would then
> call but ifdefs in main code obfuscate it. See src/util/sss_krb5.c or
> src/util/sss_ldap.c for examples.
>
> We should amend our coding guidelines to specifically advise to put
> ifdefs into a wrapper module.
>
> I know that there is the netlink module which does the same but arguably
> the netlink module is a wrapper module mostly and actually we might want
> to split it in future (feel free to file a ticket).
>
> So there should be a src/util/sss_col.c module or something like that
> that would export functions or macros with the same prototype no matter
> the libini version. That module will contain a function that will return
> ldif which can be then consumed by confdb.
>
> Also please don't use the old debug levels in new code.

Hi,

New patch is attached, it supplies src/util/sss_ini.c & 
src/util/sss_ini.h, which are exporting functions used in confdb_setup.

Please, let me know, if this is the approach you meant, and with 
comments for the coding.

Ondra


-- 
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-DB-Switch-to-new-libini_config-API.patch
Type: text/x-patch
Size: 32362 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130410/801f1103/attachment.bin>


More information about the sssd-devel mailing list