RE: [PATCH] drivers/video/ep93xx-fb.c: clean up error-handling code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2012-04-19 at 07:14 +0200, Julia Lawall wrote:
> I don't know thw impact of the ./.  I use the options
> 
> --nokeywords --nogit-fallback --subsystem --norolestats -f
> 
> julia
> 
> On Thu, 19 Apr 2012, Jingoo Han wrote:
> 
> >> -----Original Message-----
> >> From: Ryan Mallon
[]
> >> Oddly, scripts/get_maintainer.pl on this file doesn't return me, even
> >> though, according to git blame, I am the author of 90% of the commits.
> >> Should I have an entry in the MAINTAINERS file, or is
> >> scripts/get_maintainer.pl not working properly?
> >
> > There are optional differences in using scripts/get_maintainer.pl.
> > If you use './' ahead of file path, you will see your name.
> >
> > Without './' ahead of 'drivers/video/ep93xx-fb.c':
> > ./scripts/get_maintainer.pl --file drivers/video/ep93xx-fb.c
> > Florian Tobias Schandinat <FlorianSchandinat@xxxxxx> (maintainer:FRAMEBUFFER LAYER)
> > linux-fbdev@xxxxxxxxxxxxxxx (open list:FRAMEBUFFER LAYER)
> > linux-kernel@xxxxxxxxxxxxxxx (open list)
> >
> > With './' ahead of 'drivers/video/ep93xx-fb.c':
> > ./scripts/get_maintainer.pl --file ./drivers/video/ep93xx-fb.c
> > Ryan Mallon <rmallon@xxxxxxxxx> (commit_signer:2/3=67%)
> > Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> (commit_signer:1/3=33%)
> > H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> (commit_signer:1/3=33%)
> > Jesper Juhl <jj@xxxxxxxxxxxxx> (commit_signer:1/3=33%)
> > Jiri Kosina <jkosina@xxxxxxx> (commit_signer:1/3=33%)
> > linux-kernel@xxxxxxxxxxxxxxx (open list)

Prefixing "./" to the file path bypasses the normal
get_maintainers pattern check for file maintainership.

Florian Schandinat and linux-fbdev are not returned
in the second lookup above.

One of the patterns for FRAMEBUFFER is
F:	drivers/video/
which is a direct match for is ep93xx-fb.c file,
so --git is not used by default.

Adding --git may be appropriate and it also returns
Ryan's name in this case.

$ ./scripts/get_maintainer.pl -f drivers/video/ep93xx-fb.c
Florian Tobias Schandinat <FlorianSchandinat@xxxxxx> (maintainer:FRAMEBUFFER LAYER)
linux-fbdev@xxxxxxxxxxxxxxx (open list:FRAMEBUFFER LAYER)
linux-kernel@xxxxxxxxxxxxxxx (open list)

$ ./scripts/get_maintainer.pl --git -f drivers/video/ep93xx-fb.c
Florian Tobias Schandinat <FlorianSchandinat@xxxxxx> (maintainer:FRAMEBUFFER LAYER)
Ryan Mallon <rmallon@xxxxxxxxx> (commit_signer:2/3=67%)
Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> (commit_signer:1/3=33%)
H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> (commit_signer:1/3=33%)
Jesper Juhl <jj@xxxxxxxxxxxxx> (commit_signer:1/3=33%)
Jiri Kosina <jkosina@xxxxxxx> (commit_signer:1/3=33%)
linux-fbdev@xxxxxxxxxxxxxxx (open list:FRAMEBUFFER LAYER)
linux-kernel@xxxxxxxxxxxxxxx (open list)

Nominally, as framebuffer maintainer, Florian
should also have "signed-off" on these commits,
but many subsystems don't actually have a
single path or person for patches and commits.

This file came into the tree via Andrew Morton.

Another way to investigate who actually does
work on an individual file is to use the
get_maintainer.pl option "--interactive".

That can give you an output like:

*  # email/list and role:stats                                        auth sign
*  1 Florian Tobias Schandinat <FlorianSchandinat@xxxxxx>             
     maintainer:FRAMEBUFFER LAYER
*  2 Ryan Mallon <rmallon@xxxxxxxxx>                                     4    4
     commit_signer:2/3=67%,authored lines:645/650=99%,commits:4/8=50%
*  3 Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx>                       1    1
     commit_signer:1/3=33%,commits:1/8=12%
*  4 H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx>                    0    2
     commit_signer:1/3=33%,commits:2/8=25%
*  5 Jesper Juhl <jj@xxxxxxxxxxxxx>                                      0    1
     commit_signer:1/3=33%
*  6 Jiri Kosina <jkosina@xxxxxxx>                                       0    1
     commit_signer:1/3=33%
*  7 Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>                           0    2
     commits:2/8=25%
*  8 Russell King <rmk+kernel@xxxxxxxxxxxxxxxx>                          0    1
     commits:1/8=12%
*  9 linux-fbdev@xxxxxxxxxxxxxxx                                      
     open list:FRAMEBUFFER LAYER
* 10 linux-kernel@xxxxxxxxxxxxxxx                                     
     open list

About maintainership for individual files:

If anyone thinks they are the maintainer for
a particular file, generally they should submit
a patch to MAINTAINERS adding a section with
their name, maintainership level and an
appropriate file pattern.

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux