diff mbox series

[v3] utils/test-pkg: return sum of build and legal errors

Message ID 20191010102426.5452-1-heiko.thiery@gmail.com
State Accepted
Headers show
Series [v3] utils/test-pkg: return sum of build and legal errors | expand

Commit Message

Heiko Thiery Oct. 10, 2019, 10:24 a.m. UTC 
From: Heiko Thiery <heiko.thiery@kontron.com>

Signed-off-by: Heiko Thiery <heiko.thiery@kontron.com>
---
 utils/test-pkg | 2 ++
 1 file changed, 2 insertions(+)

Comments

Thomas Petazzoni Oct. 11, 2019, 7:37 a.m. UTC | #1 
On Thu, 10 Oct 2019 12:24:26 +0200
heiko.thiery@gmail.com wrote:

> From: Heiko Thiery <heiko.thiery@kontron.com>
> 
> Signed-off-by: Heiko Thiery <heiko.thiery@kontron.com>
> ---
>  utils/test-pkg | 2 ++
>  1 file changed, 2 insertions(+)

I reworked the commit title and commit log, and applied to master.
Indeed, your commit title was explain *what* is implemented, but not
why.

Best regards,

Thomas
Heiko Thiery Oct. 11, 2019, 8 a.m. UTC | #2 
> I reworked the commit title and commit log, and applied to master.
> Indeed, your commit title was explain *what* is implemented, but not
> why.

thank you
Peter Korsgaard Oct. 28, 2019, 4:36 p.m. UTC | #3 
>>>>> "heiko" == heiko thiery <heiko.thiery@gmail.com> writes:

 > From: Heiko Thiery <heiko.thiery@kontron.com>
 > Signed-off-by: Heiko Thiery <heiko.thiery@kontron.com>

Committed to 2019.02.x and 2019.08.x, thanks.
diff mbox series

Patch

diff --git a/utils/test-pkg b/utils/test-pkg
index f3b34d5d0d..a317d8c17a 100755
--- a/utils/test-pkg
+++ b/utils/test-pkg
@@ -138,6 +138,8 @@  main() {
 
     printf "%d builds, %d skipped, %d build failed, %d legal-info failed\n" \
         ${nb} ${nb_skip} ${nb_fail} ${nb_legal}
+
+    return $((nb_fail + nb_legal))
 }
 
 build_one() {