Jakub Narebski <jnareb@xxxxxxxxx> writes: > Junio C Hamano <gitster@xxxxxxxxx> writes: > >> * jc/submittingpatches (Sun Feb 3 17:02:28 2008 -0800) 3 commits >> + Documentation/SubmittingPatches: What's Acked-by and Tested-by? >> + Documentation/SubmittingPatches: discuss first then submit >> + Documentation/SubmittingPatches: Instruct how to use [PATCH] >> Subject header >> >> These I think are sensible but they did not see much discussion, >> so they are parked here for now. > > In those series I think the middle patch could be improved. I guess > that need for brevity overcame need for being explicit. I don't know > if patches meant for discussion are to be send to mailing list only, > or if the patches meant for submissions are to be sent to git mailing > list _and_ maintainer (and is it an error to send them only to the > list) from this description. Yeah, I was very unsure about the wording. What I think is the ideal patch flow is: (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). The people could include me, but that is not because I am currently the maintainer, but because I may happen to have been involved in the code you are touching. (2) You get comments and suggestions for improvements. You may even get them in a "on top of" 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 me. (5) It is merged to 'next', and cooked further and eventually graduates to 'master'. In any time between the (2)-(3) cycle, I should 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. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html