[SSSD] [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Lukas Slebodnik
ehlo, The first 4 patches are slightly modified version of Michal's patches. Mostly coding style issues, fixed reports from static analyzers + fixed small issues which would be fixed as part of review process. Last two patches are optional improvements which changes ofiginal API. It was not discu

[SSSD] Re: [PATCHES] gpo and ding libs

2016-06-22 Thread Lukas Slebodnik
On (05/05/16 16:28), Michal Židek wrote: >New patches are attached. > >SSSD patch changes: >-I changed the behavior to only try >parsing with the new flag when it >previously failed as proposed by Sumit. > >-Existence of the flag is now >checked in the GPO code, not >in util.h as Lukas suggested >

[SSSD] Re: [PATCHES] gpo and ding libs

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 09:40:47AM +0200, Lukas Slebodnik wrote: > On (05/05/16 16:28), Michal Židek wrote: > >New patches are attached. > > > >SSSD patch changes: > >-I changed the behavior to only try > >parsing with the new flag when it > >previously failed as proposed by Sumit. > > > >-Existenc

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: > ehlo, > > The first 4 patches are slightly modified version > of Michal's patches. Mostly coding style issues, fixed reports from static > analyzers + fixed small issues which would be fixed as part of review process. > > Last two

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Lukas Slebodnik
On (22/06/16 10:39), Jakub Hrozek wrote: >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: >> ehlo, >> >> The first 4 patches are slightly modified version >> of Michal's patches. Mostly coding style issues, fixed reports from static >> analyzers + fixed small issues which would be

[SSSD] Re: [PATCH] SYSDB: Fixing DB update

2016-06-22 Thread Lukas Slebodnik
On (20/06/16 14:44), Petr Cech wrote: >Hi, > >there is patch for #3023 attached. > >Reason for this patch is explained in commit message. > >I am open to discussion. In my opinion we don't need to connect sysdb pointer >with domain structure because it is not used in other upgrade functions too. >

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 10:52:34AM +0200, Lukas Slebodnik wrote: > On (22/06/16 10:39), Jakub Hrozek wrote: > >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: > >> ehlo, > >> > >> The first 4 patches are slightly modified version > >> of Michal's patches. Mostly coding style issue

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Lukas Slebodnik
On (22/06/16 11:43), Jakub Hrozek wrote: >On Wed, Jun 22, 2016 at 10:52:34AM +0200, Lukas Slebodnik wrote: >> On (22/06/16 10:39), Jakub Hrozek wrote: >> >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: >> >> ehlo, >> >> >> >> The first 4 patches are slightly modified version >> >

[SSSD] release notes for the ding-libs 0.6.0 release

2016-06-22 Thread Jakub Hrozek
Hi, I prepared very simple release notes for ding-libs 0.6.0: https://fedorahosted.org/sssd/wiki/Releases/DingNotes-0.6.0 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorah

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 11:48:20AM +0200, Lukas Slebodnik wrote: > Thank you for review. > > Feel free to change/add comments before pushing patches. I just added a simple comment that the structure field is unused at the moment. All patches are pushed to ding-libs master: * a62db8ede4deba1

[SSSD] Announcing ding-libs 0.6.0

2016-06-22 Thread Jakub Hrozek
A new version of ding-libs was released today! ding-libs, or "Ding is not GLib" is a a set of helpful libraries used by projects such as SSSD or gss-proxy. The tarball can be downloaded from: https://fedorahosted.org/sssd/wiki/Releases == Highlights == * libini_config * libini now suppor

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-22 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 04:44:05PM +0200, Jakub Hrozek wrote: > On Tue, Jun 21, 2016 at 04:15:06PM +0200, Pavel Březina wrote: > > On 06/21/2016 03:50 PM, Jakub Hrozek wrote: > > > On Mon, Jun 20, 2016 at 07:13:10PM +0200, Jakub Hrozek wrote: > > > > Hi, > > > > > > > > Pavel's sssctl tool is at:

[SSSD] Re: [PATCH] SYSDB: Fixing DB update

2016-06-22 Thread Petr Cech
On 06/22/2016 11:14 AM, Lukas Slebodnik wrote: On (20/06/16 14:44), Petr Cech wrote: >Hi, > >there is patch for #3023 attached. > >Reason for this patch is explained in commit message. > >I am open to discussion. In my opinion we don't need to connect sysdb pointer >with domain structure because