diff options
| author | Junio C Hamano <gitster@pobox.com> | 2008-02-10 14:09:52 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2008-02-16 17:33:31 -0800 |
| commit | a941fb4a43472b7b4419c6f6776749791c4495ef (patch) | |
| tree | 4a5a79441f19e8560bd2a20260c2b6b16596e10f /Documentation/SubmittingPatches | |
| parent | c11c3b56818bcaf1acea6cb2a9a68fbca644f968 (diff) | |
| download | git-a941fb4a43472b7b4419c6f6776749791c4495ef.tar.gz git-a941fb4a43472b7b4419c6f6776749791c4495ef.tar.xz | |
Documentation/SubmittingPatches - a suggested patch flow
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/SubmittingPatches')
| -rw-r--r-- | Documentation/SubmittingPatches | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches index 066129337..0e155c936 100644 --- a/Documentation/SubmittingPatches +++ b/Documentation/SubmittingPatches @@ -230,6 +230,41 @@ to modify. "Tested-by:" says the patch was tested by the person and found to have the desired effect. ------------------------------------------------ +An ideal patch flow + +Here is an ideal patch flow for this project the current maintainer +suggests to the contributors: + + (0) You come up with an itch. You code it up. + + (1) Send it to the list and cc people who may need to know about + the change. + + The people who may need to know are the ones whose code you + are butchering. These people happen to be the ones who are + most likely to be knowledgeable enough to help you, but + they have no obligation to help you (i.e. you ask for help, + don't demand). "git log -p -- $area_you_are_modifying" would + help you find out who they are. + + (2) You get comments and suggestions for improvements. You may + even get them in a "on top of your change" patch form. + + (3) Polish, refine, and re-send to the list and the people who + spend their time to improve your patch. Go back to step (2). + + (4) The list forms consensus that the last round of your patch is + good. Send it to the list and cc the maintainer. + + (5) A topic branch is created with the patch and is merged to 'next', + and cooked further and eventually graduates to 'master'. + +In any time between the (2)-(3) cycle, the maintainer may pick it up +from the list and queue it to 'pu', in order to make it easier for +people play with it without having to pick up and apply the patch to +their trees themselves. + +------------------------------------------------ MUA specific hints Some of patches I receive or pick up from the list share common |