Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Stef Walter
On 06.05.2014 01:51, Dmitri Pal wrote: > On 05/05/2014 12:28 PM, Pavel Březina wrote: >> On 05/05/2014 06:17 PM, Sumit Bose wrote: >>> On Mon, May 05, 2014 at 05:38:58PM +0200, Pavel Březina wrote: On 05/05/2014 05:25 PM, Jakub Hrozek wrote: > On Mon, May 05, 2014 at 05:08:39PM +0200, Pave

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Dmitri Pal
On 05/05/2014 12:28 PM, Pavel Březina wrote: On 05/05/2014 06:17 PM, Sumit Bose wrote: On Mon, May 05, 2014 at 05:38:58PM +0200, Pavel Březina wrote: On 05/05/2014 05:25 PM, Jakub Hrozek wrote: On Mon, May 05, 2014 at 05:08:39PM +0200, Pavel Březina wrote: On 05/05/2014 02:06 PM, Stef Walter

Re: [SSSD] [PATCH] DEBUG macro refactoring - 1.11 backport

2014-05-05 Thread Lukas Slebodnik
On (01/05/14 00:51), Nikolai Kondrashov wrote: >On 04/30/2014 11:39 PM, Nikolai Kondrashov wrote: >>On 04/30/2014 11:02 PM, Lukas Slebodnik wrote: >>>I realised that sssd cannot be built after applying the 5th and the 6th >>>patch. >>>0005-Make-DEBUG-macro-definition-variadic.patch >>>0006-Make-DE

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Pavel Březina
On 05/05/2014 06:17 PM, Sumit Bose wrote: On Mon, May 05, 2014 at 05:38:58PM +0200, Pavel Březina wrote: On 05/05/2014 05:25 PM, Jakub Hrozek wrote: On Mon, May 05, 2014 at 05:08:39PM +0200, Pavel Březina wrote: On 05/05/2014 02:06 PM, Stef Walter wrote: On 01.05.2014 17:39, Pavel Březina wro

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Sumit Bose
On Mon, May 05, 2014 at 05:38:58PM +0200, Pavel Březina wrote: > On 05/05/2014 05:25 PM, Jakub Hrozek wrote: > >On Mon, May 05, 2014 at 05:08:39PM +0200, Pavel Březina wrote: > >>On 05/05/2014 02:06 PM, Stef Walter wrote: > >>>On 01.05.2014 17:39, Pavel Březina wrote: > https://fedorahosted.org

Re: [SSSD] FYI: OTP joins in Active Directory

2014-05-05 Thread Jakub Hrozek
On Mon, May 05, 2014 at 12:52:55PM +0200, Stef Walter wrote: > Dmitri was asking me OTP joins work against Active Directory, and how > realmd and adcli accomplish them: > > http://stef.thewalter.net/2014/05/how-to-join-active-directory-domains.html > > Cheers, Nice! Would you mind sending this

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Pavel Březina
On 05/05/2014 05:25 PM, Jakub Hrozek wrote: On Mon, May 05, 2014 at 05:08:39PM +0200, Pavel Březina wrote: On 05/05/2014 02:06 PM, Stef Walter wrote: On 01.05.2014 17:39, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/2254 Lukáš already did first round of review for build and packa

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Jakub Hrozek
On Mon, May 05, 2014 at 05:08:39PM +0200, Pavel Březina wrote: > On 05/05/2014 02:06 PM, Stef Walter wrote: > >On 01.05.2014 17:39, Pavel Březina wrote: > >>https://fedorahosted.org/sssd/ticket/2254 > >> > >>Lukáš already did first round of review for build and packaging stuff. > >>Thank you, I hop

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Pavel Březina
On 05/05/2014 02:06 PM, Stef Walter wrote: On 01.05.2014 17:39, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/2254 Lukáš already did first round of review for build and packaging stuff. Thank you, I hope I have fixed all your concerns. There might be some more since I moved the libr

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Simo Sorce
On Mon, 2014-05-05 at 14:06 +0200, Stef Walter wrote: > > But, back to the point: I would caution against making it a public API > and having it become an fixture that people turn to when trying to > talk with SSSD. +1 Simo. -- Simo Sorce * Red Hat, Inc * New York _

Re: [SSSD] [PATCH] sss_config: alter configuration file

2014-05-05 Thread Lukas Slebodnik
On (05/05/14 11:36), Pavel Březina wrote: >Hi, >thank you for review. Can you please remove unneeded quotation next time? sure :-) >>>+ >>>+static errno_t >>>+sss_config_set_list(struct sss_config_ctx *ctx, >>>+const char *section, >>>+const char *option, >

Re: [SSSD] [PATCH] libsss_dbus

2014-05-05 Thread Stef Walter
On 01.05.2014 17:39, Pavel Březina wrote: > https://fedorahosted.org/sssd/ticket/2254 > > Lukáš already did first round of review for build and packaging stuff. > Thank you, I hope I have fixed all your concerns. There might be some > more since I moved the library into libsss_dbus and libsss_dbus

[SSSD] FYI: OTP joins in Active Directory

2014-05-05 Thread Stef Walter
Dmitri was asking me OTP joins work against Active Directory, and how realmd and adcli accomplish them: http://stef.thewalter.net/2014/05/how-to-join-active-directory-domains.html Cheers, Stef ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.o

Re: [SSSD] [PATCH] IFP: Per-attribute ACL for users

2014-05-05 Thread Pavel Březina
On 05/02/2014 06:34 PM, Jakub Hrozek wrote: This patch depends on all the GetUserAttr patches. Introduces a new option called user_attributes that allows to specify which user attributes are allowed to be queried from the IFP responder. By default only the default POSIX set is allowed, this opt

Re: [SSSD] [PATCH] IFP: Close memstream handle in introspect destructor

2014-05-05 Thread Pavel Březina
On 05/02/2014 06:35 PM, Jakub Hrozek wrote: Hi, attached is a small fix I found when running the sbus tests through valgrind. Ack. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-dev

Re: [SSSD] [PATCH] IFP: Add GetUserAttrs call

2014-05-05 Thread Stef Walter
On 02.05.2014 17:25, Jakub Hrozek wrote: >>> [PATCH 1/6] SBUS: two trivial style fixes SSIA >>> >>> [PATCH 2/6] SBUS: Add a convenience function Adds a convenience >>> function that constructs a DBusError internally and as such can be >>> used to mark an sbus request as failed without having to cre