Re: [U-Boot] [PATCH v2] patman: make run results better visible
Vadim, On Thu, Sep 4, 2014 at 10:45 AM, Vadim Bendebury wrote: > For an occasional user of patman some failures are not obvious: for > instance when checkpatch reports warnings, the dry run still reports > that the email would be sent. If it is not dry run, the warnings are > shown on the screen, but it is not clear that the email was not sent. > > Add some code to report failure to send email explicitly. > > Tested by running the script on a patch with style violations, > observed error messages in the script output. > > Signed-off-by: Vadim Bendebury > --- > > Changes in v2: > - modified the error message for accuracy > > tools/patman/patman.py | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) Reviewed-by: Doug Anderson ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] patman: make run results better visible
On 4 September 2014 12:57, Doug Anderson wrote: > Vadim, > > On Thu, Sep 4, 2014 at 10:45 AM, Vadim Bendebury > wrote: > > For an occasional user of patman some failures are not obvious: for > > instance when checkpatch reports warnings, the dry run still reports > > that the email would be sent. If it is not dry run, the warnings are > > shown on the screen, but it is not clear that the email was not sent. > > > > Add some code to report failure to send email explicitly. > > > > Tested by running the script on a patch with style violations, > > observed error messages in the script output. > > > > Signed-off-by: Vadim Bendebury > > --- > > > > Changes in v2: > > - modified the error message for accuracy > > > > tools/patman/patman.py | 7 ++- > > 1 file changed, 6 insertions(+), 1 deletion(-) > > Reviewed-by: Doug Anderson > Acked-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] patman: make run results better visible
Applied to u-boot-x86/buildman, thanks! ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
