summaryrefslogtreecommitdiff
path: root/README.SUBMITTING_PATCH
diff options
context:
space:
mode:
authorSascha Schumann <sas@php.net>2002-03-19 12:08:57 +0000
committerSascha Schumann <sas@php.net>2002-03-19 12:08:57 +0000
commit0459ad70a7ba320a76c52ecf18c9a4703233b3b4 (patch)
tree78ee5cddfa3a29dbe8d3d558bca07d157711ebed /README.SUBMITTING_PATCH
parent27bf1db90b6fdae2e37ef96e64748c72bd8e1f2c (diff)
downloadphp-git-0459ad70a7ba320a76c52ecf18c9a4703233b3b4.tar.gz
some typo fixing, and following The Little Book, omitting needless words
Diffstat (limited to 'README.SUBMITTING_PATCH')
-rw-r--r--README.SUBMITTING_PATCH47
1 files changed, 27 insertions, 20 deletions
diff --git a/README.SUBMITTING_PATCH b/README.SUBMITTING_PATCH
index 07b73bee9e..8e9c33f15e 100644
--- a/README.SUBMITTING_PATCH
+++ b/README.SUBMITTING_PATCH
@@ -3,7 +3,8 @@ Submitting Patch for PHP
This document describes how to submit a patch for PHP. Since you are
reading this document, you are willing to submit a patch for PHP.
-Please keep reading! Submitting a patch for PHP is easy.
+Please keep reading! Submitting a patch for PHP is easy. The hard
+part is making it acceptable for inclusion into our repository. :-)
How to create patch?
--------------------
@@ -32,6 +33,8 @@ Tips for creating patch
-----------------------
If you would like to fix multiple bugs. It is easier for us if you
could create 1 patch for 1 bug, but this is not strictly required.
+Note though that you might get little response, if your patch is
+too hard to review.
If you would like change/add many lines, it is better to ask module
maintainer and/or php-dev@lists.php.net, or pear-dev@lists.php.net if
@@ -83,34 +86,38 @@ If you are patching a module, you should also send the patch to the
maintainer. Official module maintainers are listed in EXTENSION file
in source.
-If you are pachting PEAR, send the patch to pear-dev@lists.php.net.
+If you are patching PEAR, send the patch to pear-dev@lists.php.net.
-Make sure you add "[PATCH]" prefix to mail subject. Please make sure
-attach the patch file even if the patch is really short one. If you
-create a test script for your patch, attach it to the same mail.
-Finally, explain what has been fixed/added/changed by your patch.
+Please add the prefix "[PATCH]" to your email subject and make sure
+to include the patch as a MIME attachment even if it is short.
+Test scripts should be included in the same email.
+Explain what has been fixed/added/changed by your patch.
-If you know bug ID that can be closed by your patch, please note the
-bug ID number also.
+Finally, add the bug Id(s) which can be closed by your patch, if any.
-How long it will take to get response?
+What happens after you submit your patch
--------------------------------------
-Since we are volunteers, it may take more than a few days to get
-response. If you didn't get any response in a few days, please let us
-know you have been submitted the patch, but you didn't get any
-response.
+If your patch is easy to review and has obviously no side-effects,
+it might take up to a few hours until someone commits it.
+
+Because this is a volunteer-driven effort, more complex patches will
+require more patience on your side.
+
+If you did not receive any feedback in a few days, please consider
+resubmitting the description of your patch, along-side with
+these questions:
+
+- Is my patch too hard to review? Because of which factors?
+- Should I split it up in multiple parts?
+- Are there any unwanted whitespace changes?
What happens when your patch is applied?
----------------------------------------
-When your patch is applied, it will be noted that the patch is
-submitted by you. You may see your email address in ChangeLog file. If
-your patch is important to be noted in NEWS file, your patch
-description and email address will be noted.
-
-If you would not like to be noted in ChangeLog and NEWS file, please
-let us know when you submit your patch.
+Your name will be included together with your email address in the CVS
+commit log. If your patch affects end-users, a brief description
+and your name might be added to the NEWS file.
Thank you for submitting patch for PHP!