Re: [BUG]: get-maintainers unable to respect parenthesis in subsystem name

From: Joe Perches
Date: Mon Feb 15 2016 - 00:42:55 EST


On Mon, 2016-02-15 at 10:27 +0530, Viresh Kumar wrote:
> Hi Joe,
>
> I used get-maintainers today to submit few patches for OPP framework
> and that is defined as below in MAINTAINERS:
>
> OPERATING PERFORMANCE POINTS (OPP)
> M:      Viresh Kumar <vireshk@xxxxxxxxxx>
> M:      Nishanth Menon <nm@xxxxxx>
> M:      Stephen Boyd <sboyd@xxxxxxxxxxxxxx>
> L:      linux-pm@xxxxxxxxxxxxxxx
> S:      Maintained
> T:      git git://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git
> F:      drivers/base/power/opp/
> F:      include/linux/pm_opp.h
> F:      Documentation/power/opp.txt
> F:      Documentation/devicetree/bindings/opp/
>
>
> The parenthesis in the subsystem-name causes the cc list to look like:
>
> Cc: linaro-kernel@xxxxxxxxxxxxxxxx,
> linux-pm@xxxxxxxxxxxxxxx,
> Viresh Kumar <viresh.kumar@xxxxxxxxxx>,
> Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>,
> Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>,
> Len Brown <len.brown@xxxxxxxxx>,
> linux-kernel@xxxxxxxxxxxxxxx (open list),
> linux-pm@xxxxxxxxxxxxxxx) (open list:OPERATING PERFORMANCE POINTS (OPP),
> Nishanth Menon <nm@xxxxxx>,
> Pavel Machek <pavel@xxxxxx>,
> Stephen Boyd <sboyd@xxxxxxxxxxxxxx>,
> Viresh Kumar <vireshk@xxxxxxxxxx>
>
>
>
> Look at the second linux-pm entry here, it adds a ')' at the end of
> the list's address
> and removes it from the end of the line.
>
> And so that becomes an invalid address to git-send-email.
>
> Dropping () from the subsystem name fixes it though..
>
> Thanks
>
> --
> viresh

Not what I get with current -next:

$ ./scripts/get_maintainer.pl -f Documentation/power/opp.txt 
Viresh Kumar <vireshk@xxxxxxxxxx> (maintainer:OPERATING PERFORMANCE POINTS (OPP))
Nishanth Menon <nm@xxxxxx> (maintainer:OPERATING PERFORMANCE POINTS (OPP))
Stephen Boyd <sboyd@xxxxxxxxxxxxxx> (maintainer:OPERATING PERFORMANCE POINTS (OPP))
"Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx> (supporter:SUSPEND TO RAM)
Len Brown <len.brown@xxxxxxxxx> (supporter:SUSPEND TO RAM)
Pavel Machek <pavel@xxxxxx> (supporter:SUSPEND TO RAM)
linux-pm@xxxxxxxxxxxxxxx (open list:OPERATING PERFORMANCE POINTS (OPP))
linux-kernel@xxxxxxxxxxxxxxx (open list)