Re: [Freeipa-devel] [PATCHES 0031-0032] set up a dedicated CCache file for Apache during install/upgrade

2015-04-29 Thread Martin Kosek
On 04/28/2015 05:42 PM, Martin Babinsky wrote: The attached patches address https://fedorahosted.org/freeipa/ticket/4973 and implement the solution proposed in Comment 2. Please review the hell out of them. Why did you split the work in 2 patches? It looks like you first did the first

Re: [Freeipa-devel] [PATCHES 0031-0032] set up a dedicated CCache file for Apache during install/upgrade

2015-04-29 Thread Martin Babinsky
On 04/29/2015 09:09 AM, Martin Kosek wrote: On 04/28/2015 05:42 PM, Martin Babinsky wrote: The attached patches address https://fedorahosted.org/freeipa/ticket/4973 and implement the solution proposed in Comment 2. Please review the hell out of them. Why did you split the work in 2 patches?

Re: [Freeipa-devel] [PATCHES 0031-0032] set up a dedicated CCache file for Apache during install/upgrade

2015-04-29 Thread Simo Sorce
On Wed, 2015-04-29 at 09:29 +0200, Martin Babinsky wrote: On 04/29/2015 09:09 AM, Martin Kosek wrote: On 04/28/2015 05:42 PM, Martin Babinsky wrote: The attached patches address https://fedorahosted.org/freeipa/ticket/4973 and implement the solution proposed in Comment 2. Please