[U-Boot] [PATCH] MAKEALL: Add -m option to determine maintainers

Marek Vasut marex at denx.de
Mon Mar 5 21:37:43 CET 2012


Dear Wolfgang Denk,

> Dear Marek Vasut,
> 
> In message <201203052114.10141.marex at denx.de> you wrote:
> > > 	-> ./MAKEALL -m TQM860L
> > > 	tail: cannot open `1090' for reading: No such file or directory
> > > 	TQM860L: Joakim.Tjernlund at transmode.se plagnioj at jcrosoft.com
> > > 
> > > ptyser at xes-inc.com wd at denx.de
> > > 
> > > What's this "tail" error?
> > 
> > Yes, thanks for finding this. We have multiple TQM860L in MAINTAINERS.
> 
> Umm... no, there are two entries which contain this string: "TQM860L"
> and "TQM860L_FEC", but thse are (or were) considered to be different.

Yes, fixed now.
> 
> > > 	-> ./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.
> > 
> > What's this supposed to do? You mean ./MAKEALL -m -c mpc8xx (missing -c)
> > ?
> 
> Actualy I meant to test "./MAKEALL -m 8xx" which appears to work
> mostly fine.  In any case the git error message should not be printed.

But this shouldn't print anything because board "mpc8xx" doesn't exist. Though 
this is still unfixed, V3 on the way, ignore V2 please.
> 
> 
> BTW:
> 
> -> ./MAKEALL -m AMX860
> AMX860:Joakim.Tjernlund at transmode.se ptyser at xes-inc.com
> ratbert at faraday-tech.com wd at denx.de wdenk
> 
> These entries like "wdenk" here from CVS times are really obsolete.
> Should we keep them, or introduce a cut-off-date (or -commit) ?

Found already and fixed in V2, though V3 will be necessary.
> 
> 
> Best regards,
> 
> Wolfgang Denk

Best regards,
Marek Vasut


More information about the U-Boot mailing list