[SSSD] [PATCH] CI: Remove Valgrind leak checking

Jakub Hrozek jhrozek at redhat.com
Thu Dec 4 21:59:45 UTC 2014


On Thu, Dec 04, 2014 at 09:33:49PM +0100, Lukas Slebodnik wrote:
> On (04/12/14 21:20), Jakub Hrozek wrote:
> >On Thu, Dec 04, 2014 at 06:44:33PM +0100, Lukas Slebodnik wrote:
> >> >I don't this there is as much of a technical issue with CI as process
> >> >issue.
> >> If there aren't technical issues with CI could you explain why nobody use it?
> >
> >I /do/ try to push to CI during review as
> >well, although the recent failure shows I don't do it all the time.
> >
> >But more importantly - it's triggered after every commit.
> yes but nobody cares and nobody send patches to fix issues.
> 
> There isn't patch for fixing memory leak caused by libpopt.
> The patch from this thread does not count becuase you *do* want to test memory
> leaks with valgrind.

I just sent patches for the popt issue.

I'm glad you're so passionate about the CI issues, please channel your
energy into patch review of the popt suppression.

Thanks!



More information about the sssd-devel mailing list