netdev - Re: [PATCH] MAINTAINERS: correct entries for ISDN/mISDN section

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <08d88848280f93c171e4003027644a35740a8e8e.camel@perches.com>
Date:   Sat, 01 Feb 2020 10:41:39 -0800
From:   Joe Perches <joe@...ches.com>
To:     Lukas Bulwahn <lukas.bulwahn@...il.com>,
        Karsten Keil <isdn@...ux-pingi.de>,
        Arnd Bergmann <arnd@...db.de>
Cc:     isdn4linux@...tserv.isdn4linux.de, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] MAINTAINERS: correct entries for ISDN/mISDN section

On Sat, 2020-02-01 at 13:43 +0100, Lukas Bulwahn wrote:
> Commit 6d97985072dc ("isdn: move capi drivers to staging") cleaned up the
> isdn drivers and split the MAINTAINERS section for ISDN, but missed to add
> the terminal slash for the two directories mISDN and hardware. Hence, all
> files in those directories were not part of the new ISDN/mISDN SUBSYSTEM,
> but were considered to be part of "THE REST".

Not really.

> Rectify the situation, and while at it, also complete the section with two
> further build files that belong to that subsystem.
> 
> This was identified with a small script that finds all files belonging to
> "THE REST" according to the current MAINTAINERS file, and I investigated
> upon its output.

I believe the MAINTAINERS file will be better with the
proposed patch.

Perhaps this is a defect in the small script as
get_maintainer does already show the directory and
files as being maintained.

ie: get_maintainer.pl does this:

		##if pattern is a directory and it lacks a trailing slash, add one
		if ((-d $value)) {
		    $value =~ s@([^/])$@$1/@;
		}

So:

$ ./scripts/get_maintainer.pl -f drivers/isdn/mISDN
Karsten Keil <isdn@...ux-pingi.de> (maintainer:ISDN/mISDN SUBSYSTEM)
netdev@...r.kernel.org (open list:ISDN/mISDN SUBSYSTEM)
linux-kernel@...r.kernel.org (open list)

and

$ ./scripts/get_maintainer.pl -f drivers/isdn/mISDN/dsp.h
Karsten Keil <isdn@...ux-pingi.de> (maintainer:ISDN/mISDN SUBSYSTEM)
netdev@...r.kernel.org (open list:ISDN/mISDN SUBSYSTEM)
linux-kernel@...r.kernel.org (open list)

> Fixes: 6d97985072dc ("isdn: move capi drivers to staging")

And this patch likely does not warrant a 'Fixes:' tag.

> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@...il.com>
> ---
> Arnd, please ack or even pick it.
> It is no functional change, so I guess you could simply pick in your own
> tree for minor fixes.
> 
>  MAINTAINERS | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1f77fb8cdde3..b6a0c4fa8cfd 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -8908,8 +8908,10 @@ L:	isdn4linux@...tserv.isdn4linux.de (subscribers-only)
>  L:	netdev@...r.kernel.org
>  W:	http://www.isdn4linux.de
>  S:	Maintained
> -F:	drivers/isdn/mISDN
> -F:	drivers/isdn/hardware
> +F:	drivers/isdn/mISDN/
> +F:	drivers/isdn/hardware/
> +F:	drivers/isdn/Kconfig
> +F:	drivers/isdn/Makefile
>  
>  ISDN/CMTP OVER BLUETOOTH
>  M:	Karsten Keil <isdn@...ux-pingi.de>