Dear Marek Vasut,

In message <[email protected]> you wrote:
> The -m option lists boards including their maintainers emails. There are
> multiple strategies used to retrieve these emails:
> 
> 1) Check board/<boardname> with git log and use three most recent emails
> 2) Check board/<boardname> with git log and use three most used emails
> 3) Try finding board in MAINTAINERS file and retrieve all emails from there

I think if the board can be found in MAINTAINERS, this should have
preference, i. e. always printed first.  Normaly, it should be the
only address printed.

We might also want to double-check git log based resutls against
include/configs/<name>.h - this file is way more often edited by the
maintainers.

Eventually "-m" should just print one address; we could add "-M" for all found 
addresses.

> The result is then sorted and unique results are retrieved and reported.
> 
> Signed-off-by: Marek Vasut <[email protected]>
> Cc: Wolfgang Denk <[email protected]>
> ---
>  MAKEALL |   91 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
>  1 files changed, 88 insertions(+), 3 deletions(-)

Hm... there are also problems with the code:

        -> ./MAKEALL -m TQM860L
        tail: cannot open `1090' for reading: No such file or directory
        TQM860L: [email protected] [email protected] 
[email protected] [email protected]

What's this "tail" error?

        -> ./MAKEALL -m mpc8xx
        fatal: ambiguous argument 'board/extquote': unknown revision or path 
not in the working tree.
        Use '--' to separate paths from revisions
        mpc8xx:

Doesn't work at all.



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected]
You know that feeling when you're leaning back  on  a  stool  and  it
starts to tip over? Well, that's how I feel all the time.
- Steven Wright
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to