[SSSD] [PATCH] BUILD: Add missing header file to tarball

Jakub Hrozek jhrozek at redhat.com
Mon Mar 23 16:53:59 UTC 2015


On Mon, Mar 23, 2015 at 05:03:03PM +0200, Nikolai Kondrashov wrote:
> On 03/23/2015 03:59 PM, Lukas Slebodnik wrote:
> >On (23/03/15 14:47), Pavel Reichl wrote:
> >>
> >>
> >>On 03/23/2015 02:43 PM, Lukas Slebodnik wrote:
> >>>On (23/03/15 14:18), Jakub Hrozek wrote:
> >>>>On Mon, Mar 23, 2015 at 02:07:12PM +0100, Lukas Slebodnik wrote:
> >>>>>ehlo,
> >>>>>
> >>>>>simple patch is attached.
> >>>>>
> >>>>>LS
> >>>>Hm, the default CI test doesn't run distcheck?
> >>>>
> >>>>I admit I haven't ran distcheck myself, but was relying on the CI link
> >>>>Pavel sent..
> >>>That's the reason why I do not send link to CI result as an author of the
> >>>patch.
> >>>
> >>>In my opinion it's reviewer's task.
> >>>
> >>>LS
> >>What would be different if Jakub ran CI himself?
> >>
> >If I had reviewd your patch i would have sent links to CI result which
> >run all test (including distcheck and mock build)
> 
> I tend to agree with Pavel here: trusting developer's CI links might save us
> some time. It doesn't replace the patch-specific testing, where necessary, of
> course.

The best would be if there was some tool that would grab the patches
automagically and run the CI tests..

Time to reopen the old discussion about code review tools again? :)

Part of the problem is my fault where I thought distcheck was part of
the default CI run.



More information about the sssd-devel mailing list