Re: [Freeipa-devel] [PATCHES] Re: Changes to use a single database for dogtag and IPA

2012-11-16 Thread Martin Kosek
On 11/15/2012 07:17 PM, Petr Viktorin wrote: On 11/15/2012 05:09 PM, Martin Kosek wrote: On 11/15/2012 03:19 PM, Petr Viktorin wrote: Recently, the specfile changed (dce53e4) and the patch for changed Dogtag defaults made it to master independently (91e477b). Attaching rebased patch. Note

[Freeipa-devel] [PATCH 0093] Log memory allocation failures

2012-11-16 Thread Petr Spacek
Hello, this patch adds logging to error handling code around memory allocation. -- Petr^2 Spacek From 28fc169af55a839c9772fb32b65c678c96698df9 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Fri, 16 Nov 2012 13:17:44 +0100 Subject: [PATCH] Log memory allocation failures.

Re: [Freeipa-devel] [PATCH 0093] Log memory allocation failures

2012-11-16 Thread Simo Sorce
On Fri, 2012-11-16 at 13:31 +0100, Petr Spacek wrote: +#define FILE_LINE_FUNC_MSG %-15s: %4d: %-20s +#define FILE_LINE_FUNC_ARG __FILE__, __LINE__, __func__ Wouldn't it be more compact and less error prone to do something like: #define LOG_POSITION_MSG(str, ...) \ do { \ log_error([%-15s: %4d:

Re: [Freeipa-devel] [PATCHES] Re: Changes to use a single database for dogtag and IPA

2012-11-16 Thread Martin Kosek
On 11/16/2012 11:23 AM, Martin Kosek wrote: On 11/15/2012 07:17 PM, Petr Viktorin wrote: On 11/15/2012 05:09 PM, Martin Kosek wrote: On 11/15/2012 03:19 PM, Petr Viktorin wrote: Recently, the specfile changed (dce53e4) and the patch for changed Dogtag defaults made it to master independently

Re: [Freeipa-devel] [PATCH 0093] Log memory allocation failures

2012-11-16 Thread Petr Spacek
On 11/16/2012 02:24 PM, Simo Sorce wrote: On Fri, 2012-11-16 at 13:31 +0100, Petr Spacek wrote: +#define FILE_LINE_FUNC_MSG %-15s: %4d: %-20s +#define FILE_LINE_FUNC_ARG __FILE__, __LINE__, __func__ Wouldn't it be more compact and less error prone to do something like: #define

Re: [Freeipa-devel] [PATCH] 1072 enable transaction support

2012-11-16 Thread Nalin Dahyabhai
On Thu, Nov 15, 2012 at 11:53:44PM -0500, Rob Crittenden wrote: In order for this to work you'll need to apply the last two patches (both 0001) to slapi-nis and spin it up yourself, otherwise you'll have serious deadlock issues. I know this is extra work but this patch is potentially

Re: [Freeipa-devel] [PATCH] 1072 enable transaction support

2012-11-16 Thread Rob Crittenden
Nalin Dahyabhai wrote: On Thu, Nov 15, 2012 at 11:53:44PM -0500, Rob Crittenden wrote: In order for this to work you'll need to apply the last two patches (both 0001) to slapi-nis and spin it up yourself, otherwise you'll have serious deadlock issues. I know this is extra work but this patch is

Re: [Freeipa-devel] [PATCHES] Re: Changes to use a single database for dogtag and IPA

2012-11-16 Thread Petr Viktorin
On 11/16/2012 05:17 PM, Martin Kosek wrote: On 11/16/2012 02:25 PM, Martin Kosek wrote: On 11/16/2012 11:23 AM, Martin Kosek wrote: On 11/15/2012 07:17 PM, Petr Viktorin wrote: On 11/15/2012 05:09 PM, Martin Kosek wrote: On 11/15/2012 03:19 PM, Petr Viktorin wrote: Recently, the specfile

Re: [Freeipa-devel] [PATCHES] Re: Changes to use a single database for dogtag and IPA

2012-11-16 Thread Martin Kosek
On 11/16/2012 06:46 PM, Petr Viktorin wrote: On 11/16/2012 05:17 PM, Martin Kosek wrote: On 11/16/2012 02:25 PM, Martin Kosek wrote: On 11/16/2012 11:23 AM, Martin Kosek wrote: On 11/15/2012 07:17 PM, Petr Viktorin wrote: On 11/15/2012 05:09 PM, Martin Kosek wrote: On 11/15/2012 03:19 PM,

Re: [Freeipa-devel] RFC: freeipa-asterisk plugin

2012-11-16 Thread Dmitri Pal
On 11/01/2012 10:00 AM, Loris Santamaria wrote: Hi all, we plan to write a freeIPA configuration plugin for Asterisk, aiming to be general and useful enough to be included in Fedora and EPEL, so we would like to have your input on some issues before we write any code. I wrote down the plans

Re: [Freeipa-devel] [PATCH] 1072 enable transaction support

2012-11-16 Thread Rich Megginson
On 11/15/2012 09:53 PM, Rob Crittenden wrote: This patch enables transaction support in 389-ds-base and fixes a few transaction issues within IPA. This converts parts of the password and modrnd plugins to support transactions. The password plugin still largely runs as non-transactional

Re: [Freeipa-devel] [PATCH] 1072 enable transaction support

2012-11-16 Thread Noriko Hosoi
(2012/11/16 16:09), Rich Megginson wrote: On 11/15/2012 09:53 PM, Rob Crittenden wrote: This patch enables transaction support in 389-ds-base and fixes a few transaction issues within IPA. This converts parts of the password and modrnd plugins to support transactions. The password plugin