Re: indiscriminate get_maintainer.pl usage

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Joe Perches
Date: Thursday, May 6, 2010 - 1:52 pm

On Thu, 2010-05-06 at 10:59 -0700, Davide Libenzi wrote:

That might tend to overweight cleanup against actual logic patches,
especially as the period for commits tracked is defaulted to 1 year. 


Which is kind of what activity alone does.

I think get_maintainer with --git (the default):

Without a named MAINTAINER:

For relatively active files: works moderately well.

For relatively inactive files: might work less well.

It tends to hide the original developer and show people
that submit cleanup or fix patches.

Maybe the --git portion could be reduce the number of
commit signers added when files have named MAINTAINERS.

Without named MAINTAINERS, maybe the file should be
inspected for author info and those persons added.

If the original commit is after Linus' first wholesale
commit of 2.6.12, maybe the first git author should be
added.

Dunno.  I think any of those might be slight improvements,
but get_maintainers isn't really that much of an issue
as-is.  I think it more important to get the correct
MAINTAINERS entries and file patterns.


--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] epoll: use wrapper functions, Changli Gao, (Wed May 5, 6:57 pm)
Re: indiscriminate get_maintainer.pl usage, Stefan Richter, (Wed May 5, 11:40 pm)
Re: indiscriminate get_maintainer.pl usage, Davide Libenzi, (Thu May 6, 8:42 am)
Re: indiscriminate get_maintainer.pl usage, Joe Perches, (Thu May 6, 9:52 am)
Re: indiscriminate get_maintainer.pl usage, Davide Libenzi, (Thu May 6, 10:59 am)
Re: [PATCH] epoll: use wrapper functions, Davide Libenzi, (Thu May 6, 11:47 am)
Re: [PATCH] epoll: use wrapper functions, Peter Zijlstra, (Thu May 6, 11:51 am)
Re: indiscriminate get_maintainer.pl usage, Roland Dreier, (Thu May 6, 1:13 pm)
Re: indiscriminate get_maintainer.pl usage, Joe Perches, (Thu May 6, 1:52 pm)
Re: [PATCH] epoll: use wrapper functions, Changli Gao, (Thu May 6, 7:48 pm)
Re: [PATCH] get_maintainer.pl: ignore non-maintainer tags, Florian Mickler, (Fri May 7, 12:02 am)
Re: [PATCH] get_maintainer.pl: ignore non-maintainer tags, Stefan Richter, (Fri May 7, 12:48 pm)