[U-Boot] [RFC PATCH 3/5] MAINTAINERS.boards: add initial version

Albert ARIBAUD albert.u.boot at aribaud.net
Tue Apr 22 08:42:40 CEST 2014


Hi Wolfgang,

On Mon, 21 Apr 2014 23:16:48 +0200, Wolfgang Denk <wd at denx.de> wrote:

> Dear Daniel,
> 
> In message <CACUy__UwW=4k0CwcVHhZB8JU-_Fj1cA5TNcCcCyh8E-PF9ELiQ at mail.gmail.com> you wrote:
> > 
> > > I understand your intentions, but I have to admit that I seriously
> > > dislike this approach.  It has been quite a long way to come up with
> > > boards.cfg, which would attempt to colect all relevant information for
> > > a board in a single database.  In my opinion, this is still the right
> > > way we should go: maintain all related information in a single place.
> > 
> > the main intention is to support introduction of Kconfig, which
> > eventually obsoletes the mkconfig script. This, in turn obsoletes the
> > information about arch, CPU/SOC, vendor, special config options in
> > boards.cfg. Thus boards.cfg would only contain infos about status,
> > name and mail address of board maintainers. Furthermore we still don't
> > have infos about custodians and their trees in boards.cfg. As you
> > stated in your other response the wiki page isn't a reliable source.
> > Actually we also have some quasi-official maintainers without
> > dedicated custodian trees (e.g. sandbox, driver model). Those
> > maintainers are currently not recorded at all. So it makes sense to
> > collect all those informations in one single MAINTAINERS file. Finally
> > all contributors would have more comfort in building the relevant cc
> > list for their patches.
> 
> I fully understand your intentions, and I agree with your comments
> about information missing in boards.cfg.  I will also fully agree to
> any statement that boards.cfg is not a perfect database for the kind
> of information we would like to collect.
> 
> But I still disagree with the approach taken here.  Yes, I know that
> MAINTAINERS is just following the Linux kernel example.  But I
> believe devoutly that we should strive to collect all relevant data in
> a single database (whichever form this may have) instead of spreading
> it over a number of different files.  As is, we have to add just a
> single line to boards.cfg (or, in a more general view, an atomic entry
> to a database) to add a new board.  Introducing MAINTAINERS will
> scatter information around, and it will become a permanent nightmare
> to keep information consistent: you will have to touch several files
> and always have to keep them in sync - which has never worked well.
> 
> > > In any case, scattering such data all over the place is a bad thing to
> > > do.
> > 
> > IMHO the goal should be to have one MAINTAINERS file for maintainer
> > infos and board-specific Kconfig files for all board config stuff
> > (incl. include/configs/$boardname.h).
> 
> This sounds fine, but I feel the current implementation is a step
> backwards.  It makes things worse than better.  [And I have to admit
> that I'm not fully convinced that the end goal you pattern here would
> actually work as you describe it.]
> 
> I wonder if I'm alone with my concerns?  Anybody else with comments?

I can hardly disagree with you about ( boards.cfg + MAINTAINERS.* )
being awkward to maintain, since I'm the one who merged them into a
single boards.cfg file. :)

Can'we have the maintainer(s) be part of the board config file
from the start, like a required (but possible non-editable)
configuration item, something like "CONFIG_MAINTAINER_EMAIL"?

This would, at least, keep all information for a single board
in a single place.

> Best regards,
> 
> Wolfgang Denk

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list