Re: [parisc-linux] Re: OK, let's try cleaning up another nit. Is anyone paying attention?

Russell King (rmk@arm.linux.org.uk)
Fri, 20 Apr 2001 19:50:04 +0100


On Fri, Apr 20, 2001 at 10:59:34AM -0400, Eric S. Raymond wrote:
> All right then. I'm going to send you a bunch of dead-symbol cleanup
> patches. I'll try to stay in the mainline code and out of the port
> trees. Would you please do me the kindness of telling me which ones
> can go in and which ones you think have to go through maintainers?

directly was sent separately from the other architectures, and actually
was copied to me. I'm sure that the other architecture maintainers
feel the same way, but I'll let them comment separately.

Why? Well:

- Firstly, I can apply your patch directly to my tree without having
to bother about the effects in the other architecture trees. (hence
when I resync with Linus or Alan, I don't have to go around fixing
up rejects in other architecture trees).

- Secondly, its very easy to miss stuff in the lkml hunk of email each
day when you have less than 4 hours to read it and think about it.
(note that architecture maintainers have to read mail from their
side which may not be on lkml, think about that, think about bug fixes,
possible impacts of fixes on other machines, etc etc). Therefore,
copying their email address registered in the MAINTAINER file means
that they should not overlook your patch.

- I know that Alan does take lots of patches off lkml, but I'm not sure
what his criterion is for selecting them. In the case which started
this thread off, I'm always worried that your cleanup patch would make
it in, and then cause me problems later on.

--
Russell King (rmk@arm.linux.org.uk)                The developer of ARM Linux
             http://www.arm.linux.org.uk/personal/aboutme.html

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/