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

Daniel Schwierzeck daniel.schwierzeck at gmail.com
Thu Sep 4 22:45:41 CEST 2014


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).

-- 
- Daniel


More information about the U-Boot mailing list