[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 07:02:19PM +0200, Jakub Hrozek wrote: > On Mon, Jun 27, 2016 at 05:58:49PM +0200, Michal Židek wrote: > > On 06/27/2016 04:18 PM, Jakub Hrozek wrote: > > > On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: > > > > It depends on which patches will be pushed to

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 05:58:49PM +0200, Michal Židek wrote: > On 06/27/2016 04:18 PM, Jakub Hrozek wrote: > > On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: > > > It depends on which patches will be pushed to master. > > > but I was able to apply patch from this thread with 3way

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 11:46:33AM +0200, Lukas Slebodnik wrote: > On (23/06/16 11:03), Lukas Slebodnik wrote: > >On (20/06/16 21:09), Jakub Hrozek wrote: > >>On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: > >>> ehlo, > >>> > >>> Attached is a sligtly modified version of Michal's

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 10:57:02AM +0200, Jakub Hrozek wrote: > On Tue, Jun 21, 2016 at 10:25:41AM +0200, Lukas Slebodnik wrote: > > It looks like there is an agreement that we needn't change anything in > > libini_config for this use-case. So I will focus on other ding-libs patches. > > > > BTW I

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Michal Židek
On 06/27/2016 04:18 PM, Jakub Hrozek wrote: On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: It depends on which patches will be pushed to master. but I was able to apply patch from this thread with 3way merge on Pavel's sssctl branch from fedorapeople; there weren't any conflict

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: > It depends on which patches will be pushed to master. > but I was able to apply patch from this thread with 3way merge > on Pavel's sssctl branch from fedorapeople; there weren't any conflicts The latest ones on the list. I will pu

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Lukas Slebodnik
On (27/06/16 14:00), Jakub Hrozek wrote: >On Fri, Jun 24, 2016 at 03:55:14PM +0200, Michal Židek wrote: >> On 06/24/2016 02:52 PM, Lukas Slebodnik wrote: >> > On (24/06/16 13:51), Lukas Slebodnik wrote: >> > > On (24/06/16 13:49), Lukas Slebodnik wrote: >> > > > On (24/06/16 10:29), Michal Židek wr

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Fri, Jun 24, 2016 at 03:55:14PM +0200, Michal Židek wrote: > On 06/24/2016 02:52 PM, Lukas Slebodnik wrote: > > On (24/06/16 13:51), Lukas Slebodnik wrote: > > > On (24/06/16 13:49), Lukas Slebodnik wrote: > > > > On (24/06/16 10:29), Michal Židek wrote: > > > > > On 06/24/2016 09:57 AM, Jakub H

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Michal Židek
On 06/24/2016 02:52 PM, Lukas Slebodnik wrote: On (24/06/16 13:51), Lukas Slebodnik wrote: On (24/06/16 13:49), Lukas Slebodnik wrote: On (24/06/16 10:29), Michal Židek wrote: On 06/24/2016 09:57 AM, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: On 06/23/2

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Lukas Slebodnik
On (24/06/16 13:51), Lukas Slebodnik wrote: >On (24/06/16 13:49), Lukas Slebodnik wrote: >>On (24/06/16 10:29), Michal Židek wrote: >>>On 06/24/2016 09:57 AM, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: > On 06/23/2016 09:40 PM, Jakub Hrozek wrote: >>

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Lukas Slebodnik
On (24/06/16 13:49), Lukas Slebodnik wrote: >On (24/06/16 10:29), Michal Židek wrote: >>On 06/24/2016 09:57 AM, Jakub Hrozek wrote: >>> On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: >>> > On 06/23/2016 09:40 PM, Jakub Hrozek wrote: >>> > > On Thu, Jun 23, 2016 at 09:10:34PM +0200, J

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Lukas Slebodnik
On (24/06/16 10:29), Michal Židek wrote: >On 06/24/2016 09:57 AM, Jakub Hrozek wrote: >> On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: >> > On 06/23/2016 09:40 PM, Jakub Hrozek wrote: >> > > On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: >> > > > On Thu, Jun 23, 2016

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Michal Židek
On 06/24/2016 09:57 AM, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: On 06/23/2016 09:40 PM, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: On (23/06/16 18:

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: > On 06/23/2016 09:40 PM, Jakub Hrozek wrote: > > On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: > > > On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: > > > > On (23/06/16 18:59), Michal Židek wrote: > > >

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Michal Židek
On 06/23/2016 09:40 PM, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: On (23/06/16 18:59), Michal Židek wrote: On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: On (23/06/16 18:24), Michal Židek

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: > On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: > > On (23/06/16 18:59), Michal Židek wrote: > > >On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: > > >> On (23/06/16 18:24), Michal Židek wrote: > > >> > On 06/23/2016 05

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: > On (23/06/16 18:59), Michal Židek wrote: > >On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: > >> On (23/06/16 18:24), Michal Židek wrote: > >> > On 06/23/2016 05:50 PM, Michal Židek wrote: > >> > > Lukas, > >> > > > >> > > did you h

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Lukas Slebodnik
On (23/06/16 18:59), Michal Židek wrote: >On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: >> On (23/06/16 18:24), Michal Židek wrote: >> > On 06/23/2016 05:50 PM, Michal Židek wrote: >> > > Lukas, >> > > >> > > did you have some local patches related to >> > > this patch, that you did not send here

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Michal Židek
On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: On (23/06/16 18:24), Michal Židek wrote: On 06/23/2016 05:50 PM, Michal Židek wrote: Lukas, did you have some local patches related to this patch, that you did not send here? Because I can not apply the patch on current master and the changes in

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Lukas Slebodnik
On (23/06/16 18:24), Michal Židek wrote: >On 06/23/2016 05:50 PM, Michal Židek wrote: >> Lukas, >> >> did you have some local patches related to >> this patch, that you did not send here? >> >> Because I can not apply the patch on current >> master and the changes in the context of >> your patch

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Michal Židek
On 06/23/2016 05:50 PM, Michal Židek wrote: Lukas, did you have some local patches related to this patch, that you did not send here? Because I can not apply the patch on current master and the changes in the context of your patch seem like you did something with default config file path? Slig

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Michal Židek
Lukas, did you have some local patches related to this patch, that you did not send here? Because I can not apply the patch on current master and the changes in the context of your patch seem like you did something with default config file path? Slightly modified version that applies on current

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Lukas Slebodnik
On (23/06/16 11:51), Jakub Hrozek wrote: >On Thu, Jun 23, 2016 at 11:46:33AM +0200, Lukas Slebodnik wrote: >> On (23/06/16 11:03), Lukas Slebodnik wrote: >> >On (20/06/16 21:09), Jakub Hrozek wrote: >> >>On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: >> >>> ehlo, >> >>> >> >>> At

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 11:46:33AM +0200, Lukas Slebodnik wrote: > On (23/06/16 11:03), Lukas Slebodnik wrote: > >On (20/06/16 21:09), Jakub Hrozek wrote: > >>On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: > >>> ehlo, > >>> > >>> Attached is a sligtly modified version of Michal's

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Lukas Slebodnik
On (23/06/16 11:03), Lukas Slebodnik wrote: >On (20/06/16 21:09), Jakub Hrozek wrote: >>On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: >>> ehlo, >>> >>> Attached is a sligtly modified version of Michal's patch. >> >>The same patch is attached twice. Was it by accident or did you

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Lukas Slebodnik
On (20/06/16 21:09), Jakub Hrozek wrote: >On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: >> ehlo, >> >> Attached is a sligtly modified version of Michal's patch. > >The same patch is attached twice. Was it by accident or did you mean to >send two patches? > >> I fixed few coding

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-21 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 10:25:41AM +0200, Lukas Slebodnik wrote: > It looks like there is an agreement that we needn't change anything in > libini_config for this use-case. So I will focus on other ding-libs patches. > > BTW I need to update manual pages to reflect this change. > Please file a t

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-21 Thread Lukas Slebodnik
On (20/06/16 16:13), Stephen Gallagher wrote: >On 06/20/2016 03:09 PM, Jakub Hrozek wrote: >> On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: >>> ehlo, >>> >>> Attached is a sligtly modified version of Michal's patch. >> >> The same patch is attached twice. Was it by accident or d

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-20 Thread Stephen Gallagher
On 06/20/2016 03:09 PM, Jakub Hrozek wrote: > On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: >> ehlo, >> >> Attached is a sligtly modified version of Michal's patch. > > The same patch is attached twice. Was it by accident or did you mean to > send two patches? > >> I fixed few

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-20 Thread Jakub Hrozek
On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: > ehlo, > > Attached is a sligtly modified version of Michal's patch. The same patch is attached twice. Was it by accident or did you mean to send two patches? > I fixed few coding style issues + added missing creation of directory