[SSSD] Re: [PATCH] UTILS: Fixing duplication of pid file declaration

2016-08-09 Thread Jakub Hrozek
On Tue, Aug 09, 2016 at 11:27:12AM +0200, Jakub Hrozek wrote: > On Fri, Aug 05, 2016 at 02:49:59PM +0200, Petr Cech wrote: > > On 08/04/2016 11:06 AM, Jakub Hrozek wrote: > > > On Thu, Aug 04, 2016 at 08:41:34AM +0200, Fabiano Fidêncio wrote: > > > > On Tue, Jul 26, 2016 at 4:01 PM, Petr Cech

[SSSD] Re: [PATCH] UTILS: Fixing duplication of pid file declaration

2016-08-09 Thread Jakub Hrozek
On Fri, Aug 05, 2016 at 02:49:59PM +0200, Petr Cech wrote: > On 08/04/2016 11:06 AM, Jakub Hrozek wrote: > > On Thu, Aug 04, 2016 at 08:41:34AM +0200, Fabiano Fidêncio wrote: > > > On Tue, Jul 26, 2016 at 4:01 PM, Petr Cech wrote: > > > > Hello list, > > > > > > > > attached

[SSSD] Re: [PATCH] UTILS: Fixing duplication of pid file declaration

2016-08-05 Thread Petr Cech
On 08/04/2016 11:06 AM, Jakub Hrozek wrote: On Thu, Aug 04, 2016 at 08:41:34AM +0200, Fabiano Fidêncio wrote: On Tue, Jul 26, 2016 at 4:01 PM, Petr Cech wrote: Hello list, attached patch fixes duplication of pid file declaration. I hope that the util/util.h is the right

[SSSD] Re: [PATCH] UTILS: Fixing duplication of pid file declaration

2016-08-04 Thread Jakub Hrozek
On Thu, Aug 04, 2016 at 08:41:34AM +0200, Fabiano Fidêncio wrote: > On Tue, Jul 26, 2016 at 4:01 PM, Petr Cech wrote: > > Hello list, > > > > attached patch fixes duplication of pid file declaration. I hope that the > > util/util.h is the right place for it. Another opinion are

[SSSD] Re: [PATCH] UTILS: Fixing duplication of pid file declaration

2016-08-04 Thread Fabiano Fidêncio
On Tue, Jul 26, 2016 at 4:01 PM, Petr Cech wrote: > Hello list, > > attached patch fixes duplication of pid file declaration. I hope that the > util/util.h is the right place for it. Another opinion are welcome. LGTM! I'd wait till someone else reviews the patch, in case you

[SSSD] Re: [PATCH] UTILS: Fixing duplication of pid file declaration

2016-08-03 Thread Petr Cech
bump -- Petr^4 Čech ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org