Hi Roger,

On Wed, 20 Aug 2014 01:13:50 +0200
Roger Meier <ro...@bufferoverflow.ch> wrote:

> Signed-off-by: Roger Meier <ro...@bufferoverflow.ch>
> CC: Masahiro Yamada <yamad...@jp.panasonic.com>
> CC: Tom Rini <tr...@ti.com>
> CC: Wolfgang Denk <w...@denx.de>


Most people use 'Cc:' instead of 'CC:'.


$ git log v2013.01..v2014.07 | grep 'Cc:' | wc
   2007    8047   87385
$ git log v2013.01..v2014.07 | grep 'CC:' | wc
    147     598    6443




> diff --git a/Makefile b/Makefile
> index 666d291..6679408 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1285,7 +1285,8 @@ distclean: mrproper
>               -o -name '*.bak' -o -name '#*#' -o -name '.*.orig' \
>               -o -name '.*.rej' -o -name '*%' -o -name 'core' \
>               -o -name '*.pyc' \) \
> -             -type f -print | xargs rm -f
> +             -type f -print | xargs rm -f \
> +     @rm -f boards.cfg
>  
>  backup:
>       F=`basename $(srctree)` ; cd .. ; \


No.  You missed one of my comments against v2.

Please remove the backslash after "xargs rm -f".


Here, you are doing

@find ... | xargs rm -f @rm -f boards.cfg


that is, you are trying to delete
'@rm' as well as 'boards.cfg' if they exist.


Besides, you also need to fix buildman
as Simon suggested.


Please also cc Simon when you post the next version.




Best Regards
Masahiro Yamada

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to