patch review proceedings (was: Re: [MPlayer-dev-eng] [PATCH] make cache2.c readable)

[Attila Kinali]

On Mon, 11 Sep 2006 13:39:04 +0300
Uoti Urpala <uoti.urpala at pp1.inet.fi> wrote:

> what I said in the earlier mail: patches to code/features that have
> active maintainers get applied while other patches often won't unless
> they're completely trivial/obvious, and the proportion of unnecessarily
> dropped patches is more a function of which code parts have active
> maintainers rather than of how many "reviewers" there are.

Which boils down to something i mentioned quite a few times
over the last years (ok, not that actively the last 2-3y):

_We_need_more_maintainers_.

If you get people willing to work on and continue to maintain
some code. Give them svn write access. Quite a few of the
active maintainers these days started with being asked
"would you like to maintain this or that code?"

IMHO we should have a public list of unmaintained code
on the webpage where people could see it, and if they
feel like it apply for maintainership. Of course, the
first few patches would need to be reviewed by a mentor,
but if the guy is not all talk then our rules shouldnt
be that hard to learn and follow.

Here a short list of currently unmaintained code
extracted from MAINTAINERS (and thus incomplete):

Debian packageing
DVD/VOB subtitles
MPlayer core
MEncoder core
GUI
mp3lib
liba52
libmpeg2
libmpdvdkit2
libfaad2
vo_dga
vo_dxr3
vo_mpegpes
vo_xvidix
half of the vidix drivers
most ao modules (it's actualy simpler to count the ones maintained)

This does not include all files that are not listed in here
(i guess only half of the code is listed in MAINTAINERS),
neither does it list the code that is "maintained" by people
who disapeared or are hardly around (like me).


			Attila Kinali

-- 
egp ist vergleichbar mit einem ikea bausatz fuer flugzeugtraeger
			-- reeler in +kaosu