Re: [389-devel] Please review: fix compiler warning

2015-11-03 Thread Noriko Hosoi
On 11/03/2015 10:42 AM, Martin Basti wrote: On 03.11.2015 19:30, Noriko Hosoi wrote: Thanks for your patch, Martin. Looks good to me. "ack". Which version is your patch supposed to be in? Just master is fine? --noriko Thank you, master is fine. Pushed to master: aff9705..0d1ce9d master

Re: [389-devel] Please review: fix compiler warning

2015-11-03 Thread Noriko Hosoi
Thanks for your patch, Martin. Looks good to me. "ack". Which version is your patch supposed to be in? Just master is fine? --noriko On 11/03/2015 10:24 AM, Martin Basti wrote: Hello all, the gcc compiler returns warning during compiling of the freeIPA project: slapi-plugin.h:6149, GNU C

Re: [389-devel] Please review: fix compiler warning

2015-11-03 Thread Martin Basti
On 03.11.2015 19:30, Noriko Hosoi wrote: Thanks for your patch, Martin. Looks good to me. "ack". Which version is your patch supposed to be in? Just master is fine? --noriko Thank you, master is fine. Martin On 11/03/2015 10:24 AM, Martin Basti wrote: Hello all, the gcc compiler

Re: [389-devel] Please review ticket 47976: deadlock in mep delete post op

2015-11-03 Thread Ludwig Krispenz
Hi Thierry, we already had started to discuss on IRC, but here are my thoughts again. Is it necessary to explicitely set the txn in the plugin ? The txn will be found when ldbm_back_delete() does dblayer_txn_begin(9 and it checks the per thread stack of txns. In my opinion the real problem is

[389-devel] Please review ticket 47976: deadlock in mep delete post op

2015-11-03 Thread thierry bordaz
ticket https://fedorahosted.org/389/ticket/47976 fix https://fedorahosted.org/389/attachment/ticket/47976/0001-Ticket-47976-deadlock-in-mep-delete-post-op.patch test case: https://fedorahosted.org/389/attachment/ticket/47976/0002-Ticket-47976-test-case.patch -- 389-devel mailing list