From a7244e6ab257058669924cb5cdc14a494ea26c0b Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Tue, 29 Jul 2008 22:25:02 -0500 Subject: Further updates to submitting-patches Xavier started the job, this should clean it up and get it up to date that final bit. Signed-off-by: Dan McGee --- submitting-patches | 49 +++++++++++++++++++++++++++++++++++-------------- 1 file changed, 35 insertions(+), 14 deletions(-) diff --git a/submitting-patches b/submitting-patches index 452a1b85..e6853c5f 100644 --- a/submitting-patches +++ b/submitting-patches @@ -1,22 +1,33 @@ Pacman - Submitting Patches =========================== -This document is here mainly to make my job easier, and is more of a guideline, -and not a strict set of rules. Please try to follow as much as you can. +This document is here mainly to make the job of those who review patches +easier and is more of a guideline and not a strict set of rules. However, +please try to follow as much as you can. NOTE: Some of this is paraphrased from the kernel documentation's "SubmittingPatches" file. +Getting the most recent source +------------------------------ + +Patches need to be submitted in GIT format and are best if they are against the +latest version of the code. There are several helpful tutorials for getting +started with GIT if you have not worked with it before. + +* http://www.kernel.org/pub/software/scm/git/docs/gittutorial.html +* http://wiki.archlinux.org/index.php/Super_Quick_Git_Guide + +The pacman code can be fetched using the following command: + + git clone git://projects.archlinux.org/pacman.git + + Creating your patch ------------------- -Patches need to be submitted in GIT format. So for getting started, you will -have to read some git guides first, to learn how to fetch pacman git repo, how -to configure your name and email adress, how to create a branch, a commit, and -finally your patch. - -- -* use git commit -s for creating a commit of your changes. +* use `git commit -s` for creating a commit of your changes. The -s allows you to credit yourself by adding a "Signed Off By" line to indicate who has "signed" the patch - who has approved it. @@ -28,11 +39,11 @@ address if you're afraid of spam. * Describe your patch. -It helps if you describe the changes of the patch in the git commit log. -This allows others to see what you intended so as to compare it to -what was actually done, and allows better feedback. +It helps if you describe the overview and goals of the patch in the git commit +log. This allows others to see what you intended so as to compare it to what +was actually done, and allows better feedback. -* Use git format-patch to create patches. +* Use `git format-patch` to create patches. Your commit message will be shown above the patch by default when you will use `git-format-patch`, including the signoff line. @@ -50,11 +61,12 @@ will get feedback, and let me know the details of your patch. * No MIME, no links, no compression, no attachments. Just plain text. Patches should be contained in the actual body of the email. There are many -reasons for this. Firstly, it makes them easier to read with any mail reader, +reasons for this. First, it makes them easier to read with any mail reader, it allows easier review "at a glance", and most importantly, it allows people to comment on exact lines of the patch in reply emails. -git send-email allows you to send git formatted patches in plain text easily. +`git send-email` allows you to send git formatted patches in plain text easily +and is the preferred method for submission to the mailing list. -- @@ -70,6 +82,15 @@ We welcome most submissions here, and some may take a bit longer to get looked over than others. If you think your patch got lost in the shuffle, send another email to the list in reply to the original asking if anyone has looked at it yet. + +* Respond to feedback + +When you do get feedback, it usually merits a response, whether this be a +resubmit of the patch with corrections or a follow-up email asking for +clarifications. When neither of these occurs, don't expect your patch to see +further review. The all-volunteer staff don't have time to fix up patches that +aren't their own. + -- ///// -- cgit v1.2.3-70-g09d2