[SSSD] [PATCH] libsss_dbus

Sumit Bose sbose at redhat.com
Thu May 22 09:07:51 UTC 2014


On Mon, May 19, 2014 at 10:40:40PM +0200, Pavel Březina wrote:
> On 05/16/2014 03:32 PM, Jakub Hrozek wrote:
> >On Fri, May 16, 2014 at 02:54:55PM +0200, Pavel Březina wrote:
> >>On 05/16/2014 02:04 PM, Jakub Hrozek wrote:
> >>>On Thu, May 15, 2014 at 09:23:46PM +0200, Pavel Březina wrote:
> >>>>On 05/15/2014 09:09 PM, Pavel Březina wrote:
> 

...


> All concerns should be fixed (but the memory leak in test). I also changed
> the prefix to sss_sifp as Sumit asked (sending as separate patch).

Thank you, imo it more looks much more consistent.

Please find attached a small patch which should illustrate my other
suggestion to remove the build time DBus dependency for the user of the
library. Doc fixes and a free function for the new struct are still
missing, the patch should just illustrate the idea. If you think this
makes sense, I can add the missing parts if you like.

With this the dbus-1 requirement in the pc file can be removed as well
because the runtime dependency for DBus will be solved by package
management. Btw, I think there is an issue in the current pc file, both
requirements should be in a single line.

bye,
Sumit

> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel




More information about the sssd-devel mailing list