On Thu, 19 Sep 2019, Geert Uytterhoeven <[email protected]> wrote:
> On Thu, Sep 19, 2019 at 8:57 AM Dan Carpenter <[email protected]>
> wrote:
>> On Wed, Sep 18, 2019 at 10:57:28AM -0300, Mauro Carvalho Chehab wrote:
>> When I sent a patch, I use get_maintainer.pl then I add whoever the
>> wrote the commit from the Fixes tag. Then I remove Colin King and Kees
>> Cook from the CC list because they worked all over the tree and I know
>> them. I also normally remove LKML if there is another mailing list but
>> at least one subsystem uses LKML for patchwork so this isn't safe.
>>
>> So the safest instructions are "Use get_matainer.pl and add the person
>> who wrote the commit in the Fixes tag".
>
> Better: perhaps get_maintainer.pl can be taught to add the author of the
> commit pointed to by the Fixes tag, if present?
The drm maintainer tools [1] have that, with Cc's and reviewers picked
up, and appropriate Cc: stable added. On a random commit from v5.3:
$ dim fixes 61f7f7c8f978b1c0d80e43c83b7d110ca0496eb4
Fixes: 61f7f7c8f978 ("gpiolib: acpi: Add gpiolib_acpi_run_edge_events_on_boot
option and blacklist")
Cc: [email protected]
Cc: Daniel Drake <[email protected]>
Cc: Ian W MORRISON <[email protected]>
Cc: Hans de Goede <[email protected]>
Cc: Mika Westerberg <[email protected]>
Cc: Andy Shevchenko <[email protected]>
Cc: Linus Walleij <[email protected]>
Cc: Bartosz Golaszewski <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: <[email protected]> # v5.3+
I'm sure it could be massively improved, but it does give you the
initial list that's easy to trim.
BR,
Jani.
[1] https://gitlab.freedesktop.org/drm/maintainer-tools/blob/master/dim#L2398
--
Jani Nikula, Intel Open Source Graphics Center
