From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Sun, 4 Feb 2018 06:39:50 -0700
Subject: [U-Boot] [PATCH 2/4] buildman: add option -E for treating
 compiler warnings as errors
In-Reply-To: <20180125172127.17825-3-daniel.schwierzeck@gmail.com>
References: <20180125172127.17825-1-daniel.schwierzeck@gmail.com>
 <20180125172127.17825-3-daniel.schwierzeck@gmail.com>
Message-ID: <CAPnjgZ0NvwO=mKNW8RKuFKagx+GEYVCznOXyhu6-tpa=b6GwWg@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

`On 25 January 2018 at 10:21, Daniel Schwierzeck
<daniel.schwierzeck@gmail.com> wrote:
> Add a new option '-E' for treating all compiler warnings as errors.
> Eventually this will pass 'W=err' to Kbuild.

nit: I read 'eventually' as 'in a future patch'

How about 'This will cause buildman to pass ...' ?

>
> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> ---
>
>  tools/buildman/builder.py       | 5 ++++-
>  tools/buildman/builderthread.py | 2 ++
>  tools/buildman/cmdline.py       | 2 ++
>  tools/buildman/control.py       | 3 ++-
>  4 files changed, 10 insertions(+), 2 deletions(-)
>

Reviewed-by: Simon Glass <sjg@chromium.org>