[Buildroot] [PATCH] .gitlab-ci.yml: add check-DEVELOPERS test

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sat Jul 1 07:39:21 UTC 2017


Hello,

On Sat, 1 Jul 2017 09:36:02 +0200, Arnout Vandecappelle wrote:

> >> +check-DEVELOPERS:
> >> +    script:
> >> +        - PYTHONPATH=$PWD/support/scripts
> >> +                python -c 'from getdeveloperlib import parse_developers; parse_developers()' |
> >> +                grep . && exit 1  
> > 
> > Isn't "./support/scripts/get-developer -c" doing the right thing?  
> 
>  The problem with that is that it produces a lot of output, and the warnings are
> at the top. So I would have to grep for WARNING. But some of the possible errors
> don't have a WARNING in front of them. So I started out changing the script to
> add WARNING for those as well. But then I thought: keep things simple, just call
> parse_developers directly.
> 
>  Perhaps it would be a little more transparent to add a __main__ to
> getdeveloperlib and then call it with "python support/scripts/getdeveloperlib.py".

Or a proper option to get-developers that does whatever check you want?

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


More information about the buildroot mailing list