Re: [Freeipa-devel] [PATCH] 0049 Remove workaround for CA running check

2016-01-26 Thread Jan Cholasta
On 21.1.2016 14:10, Martin Basti wrote: On 20.01.2016 10:52, Fraser Tweedale wrote: On Wed, Jan 20, 2016 at 09:30:29AM +0100, Martin Kosek wrote: On 01/20/2016 08:45 AM, Fraser Tweedale wrote: The attached patch removes a workaround introduced as part of

Re: [Freeipa-devel] [PATCH] 0049 Remove workaround for CA running check

2016-01-21 Thread Martin Basti
On 20.01.2016 10:52, Fraser Tweedale wrote: On Wed, Jan 20, 2016 at 09:30:29AM +0100, Martin Kosek wrote: On 01/20/2016 08:45 AM, Fraser Tweedale wrote: The attached patch removes a workaround introduced as part of https://fedorahosted.org/freeipa/ticket/4676. Alternatively, if we want to

Re: [Freeipa-devel] [PATCH] 0049 Remove workaround for CA running check

2016-01-20 Thread Martin Kosek
On 01/20/2016 08:45 AM, Fraser Tweedale wrote: > The attached patch removes a workaround introduced as part of > https://fedorahosted.org/freeipa/ticket/4676. > > Alternatively, if we want to keep the "workaround" I will submit a > different patch that removes unused code and FIXME comments :) >

Re: [Freeipa-devel] [PATCH] 0049 Remove workaround for CA running check

2016-01-20 Thread Fraser Tweedale
On Wed, Jan 20, 2016 at 09:30:29AM +0100, Martin Kosek wrote: > On 01/20/2016 08:45 AM, Fraser Tweedale wrote: > > The attached patch removes a workaround introduced as part of > > https://fedorahosted.org/freeipa/ticket/4676. > > > > Alternatively, if we want to keep the "workaround" I will

Re: [Freeipa-devel] [PATCH] 0049 Remove workaround for CA running check

2016-01-20 Thread Fraser Tweedale
On Wed, Jan 20, 2016 at 07:52:32PM +1000, Fraser Tweedale wrote: > Good pickup on the curl dependency; indeed it is no longer needed. > Updated patch attached. > Whups, that was same patch, different name. *Here* is the new patch. From ba5750b7a805841abd8d4795d9c4bcec2a3518a0 Mon Sep 17 00:00:00

[Freeipa-devel] [PATCH] 0049 Remove workaround for CA running check

2016-01-19 Thread Fraser Tweedale
The attached patch removes a workaround introduced as part of https://fedorahosted.org/freeipa/ticket/4676. Alternatively, if we want to keep the "workaround" I will submit a different patch that removes unused code and FIXME comments :) Cheers, Fraser From