[Buildroot] [PATCH v2] .gitlab-ci.yml: simplify (and fix) check-DEVELOPERS

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sat Jul 1 17:44:47 UTC 2017


Hello,

On Sat, 1 Jul 2017 19:28:17 +0200, Arnout Vandecappelle
(Essensium/Mind) wrote:
> The check-DEVELOPERS test would always fail, because && will return
> the exit code of the left-hand side if not 0, so it *always* returns
> 1.
> 
> But in fact, the test can be simplified quite a lot: we can run
> getdeveloper directly, without arguments. That should just print
> 'No action specified', but it will also print any syntax errors it
> encountered. So we can just 'grep -v' the expected output away; if
> there is anything more, grep will return 0, but if there is nothing
> more, it will return 1. So we just need to invert the result.
> 
> Incidentally, this also fixes the incorrect remaining reference to
> support/scripts.
> 
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
> ---
>  .gitlab-ci.yml    | 4 +---
>  .gitlab-ci.yml.in | 4 +---
>  2 files changed, 2 insertions(+), 6 deletions(-)

Applied to master, thanks.

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



More information about the buildroot mailing list