[U-Boot] [PATCH v3 0/4] introduce get_maintainer.pl

[Simon Glass]

Hi Daniel,


On 4 September 2014 14:45, Daniel Schwierzeck <daniel.schwierzeck at gmail.com>
wrote:

> Hi Simon,
>
> 2014-09-04 1:34 GMT+02:00 Simon Glass <sjg at chromium.org>:
> > Hi Daniel,
> >
> > On 31 July 2014 18:24, Daniel Schwierzeck <daniel.schwierzeck at gmail.com>
> wrote:
> >>
> >> This series imports get_maintainer.pl from kernel and reintroduce
> >> the MAINTAINERS file in the according format. Currently one have to
> >> manually grep all infos about board maintainers or custodians from
> >> various sources like git log, wiki or boards.cfg. get_maintainer.pl
> >> makes this task much easier. Also patman is already able to use it
> >> for cc-cmd.
> >>
> >> The MAINTAINERS file contains all custodians and is hand-written based
> >> on infos from wiki [1] and u-boot git forks [2]. Thus the source files
> >> infos (F:) are not complete yet. Furthermore status (S:) and
> >> maintainership (M:) for some custodian trees are probably wrong.
> >> Any help in completing and fixing that is appreciated.
> >>
> >> The get_maintainers.pl script is modified to scan multiple
> >> MAINTAINERS files in the board directory. This is required
> >> with the switch to Kconfig which adds a MAINTAINERS file per board.
> >>
> >> Examples:
> >>
> >>  $ ./scripts/get_maintainer.pl --nogit-fallback -f board/sandbox/
> >>    Simon Glass <sjg at chromium.org> (maintainer:SANDBOX BOARD)
> >>    u-boot at lists.denx.de (open list)
> >>
> >>  $ ./scripts/get_maintainer.pl --nogit-fallback -f arch/sandbox/
> >>    Simon Glass <sjg at chromium.org> (maintainer:SANDBOX)
> >>    u-boot at lists.denx.de (open list)
> >>
> >>  $ ./scripts/get_maintainer.pl --nogit-fallback -f
> board/sandburst/metrobox/
> >>    Travis Sawyer <travis.sawyer at sandburst.com> (orphan (since
> 2014-03):METROBOX BOARD)
> >>    u-boot at lists.denx.de (open list)
> >>
> >> This series is also available at git://
> git.denx.de/u-boot-mips.git/maintainers_v3.
> >
> > Patman now picks this up which is great. But it does sometimes produce
> > a long list of maintainers. I wonder if we could have an option to
> > turn it off (perhaps -m)?
> >
> > Regards,
> > Simon
>
> I think we should generally adapt the options for get_maintainers.pl
> patman uses to not include all committers by default. Either we could
> completely disable git (--nogit-fallback). Or we could increase the
> minimum percentage of commits required (--git-min-percent 10).
>

OK, so long as there is a way to also make get_maintainers.pl a nop, then
that is fine with me.

Regards,
Simon