commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r1042908 - /commons/proper/commons-site/trunk/src/site/xdoc/patches.xml
Date Tue, 07 Dec 2010 04:16:47 GMT
Author: sebb
Date: Tue Dec  7 04:16:46 2010
New Revision: 1042908

Patches should be relative to the project root.


Modified: commons/proper/commons-site/trunk/src/site/xdoc/patches.xml
--- commons/proper/commons-site/trunk/src/site/xdoc/patches.xml (original)
+++ commons/proper/commons-site/trunk/src/site/xdoc/patches.xml Tue Dec  7 04:16:46 2010
@@ -67,7 +67,7 @@
   Please try and restrict patches to the minimum necessary to implement the change.
-  If there are a lot of irrelevant formatting or other changes, it make it much harder to
+  If there is a lot of irrelevant formatting or other changes, it makes it much harder to
   the patch, and it may be rejected.
@@ -105,6 +105,15 @@
   The patch should be in unified format. You can create a patch in this format (from Subversion)
by using:
   <source>svn diff <em>File</em> > <em>patchfile</em></source><br/>
+  </p>
+  <p>
+  The patch should be created from the project root, i.e. the directory which contains the
src/ folder
+  and the main Maven pom.xml file. This ensures that the proper relative path names are included,
+  makes it easier to apply the patch to the correct file (there can be several files with
the same name in a project).
+  If using Eclipse to create the patch, set "Patch Root" to "Project" - not the default "Workspace".
+  [Workspace-relative patches are not portable unless exactly the same project names are
+  </p>
+  <p>
   Try to give your patch files meaningful names. This makes it easier for developers who
need to 
   apply a number of different patches. 

View raw message