* [PATCH] Point out merged submodule maintainers as better addresses for patches
@ 2007-12-27 4:12 Shawn O. Pearce
2007-12-27 4:37 ` Junio C Hamano
0 siblings, 1 reply; 2+ messages in thread
From: Shawn O. Pearce @ 2007-12-27 4:12 UTC (permalink / raw)
To: Junio C Hamano, paulus; +Cc: git
Junio doesn't directly manage the git-gui or gitk-git subdirectories
within git.git; these are currently managed by other individuals
and then periodically merged to git.git. Users often send patches
to Junio for these directories when they should be directed at the
current maintainer instead, so we should point out these special
cases in the SubmittingPatches documentation.
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
---
Just a suggestion. Might help people who are new to the Git
community to get involved.
Documentation/SubmittingPatches | 8 +++++++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index de08d09..b7c886c 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -34,10 +34,16 @@ Checklist (and a short version for the impatient):
- if your name is not writable in ASCII, make sure that
you send off a message in the correct encoding.
- send the patch to the list (git@vger.kernel.org) and the
- maintainer (gitster@pobox.com). If you use
+ active maintainer (see below). If you use
git-send-email(1), please test it first by sending
email to yourself.
+ Active maintainers:
+
+ - git-gui/ submodule: spearce@spearce.org
+ - gitk-git/ submodule: paulus@samba.org
+ - Everything else: gitster@pobox.com
+
Long version:
I started reading over the SubmittingPatches document for Linux
--
1.5.4.rc2.1097.gb6e0d
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Point out merged submodule maintainers as better addresses for patches
2007-12-27 4:12 [PATCH] Point out merged submodule maintainers as better addresses for patches Shawn O. Pearce
@ 2007-12-27 4:37 ` Junio C Hamano
0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2007-12-27 4:37 UTC (permalink / raw)
To: Shawn O. Pearce; +Cc: paulus, git
"Shawn O. Pearce" <spearce@spearce.org> writes:
> Junio doesn't directly manage the git-gui or gitk-git subdirectories
> within git.git; these are currently managed by other individuals
> and then periodically merged to git.git. Users often send patches
> to Junio for these directories when they should be directed at the
> current maintainer instead, so we should point out these special
> cases in the SubmittingPatches documentation.
This is a good start, but I think it would be good to mention
that contributors should at least Cc: the people who have been
heavily involved in the past in the surrounding area of the code
or documentation (not necessarily the last person who touched
the area) the patch attempts to improve and/or enhance.
Parts of the system are "actively managed" by people other than
me even though they aren't merged with the subtree strategy.
The areas I mentioned in MaintNotes (in 'todo') are:
- git-svn is generally deferred to Eric, unless the patch is
truly trivial.
- Documentation/user-manual.txt is primarily JBF's bailiwick.
- Nico is the guy around pack generation.
- Jakub, Luben and Pasky are more familiar than I am with
gitweb.
In fact, I would rather see any nontrivial patch to be first
sent "To: " such people with "Cc: " the list. After it has been
improved into a good enough shape in the discussion that follows
(i.e. success stories to back it up, and without regression
reports and objections), I can pick up the last round directly
from the list, or the active maintainer in the area (if exists;
it might be a better wording to call them "subsystem
maintainers") can forward it to me to apply after a final
submission is made. Sending the final submission to the
subsystem maintainer who forwards it to the higher level is what
the kernel folks regularly do, but I do not think we are large
enough to require such a formality.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-12-27 4:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-12-27 4:12 [PATCH] Point out merged submodule maintainers as better addresses for patches Shawn O. Pearce
2007-12-27 4:37 ` Junio C Hamano
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).