Author: sebb Date: Thu Nov 22 10:59:47 2012 New Revision: 1412496 URL: http://svn.apache.org/viewvc?rev=1412496&view=rev Log: Clarify that patch file names should have extensions
Modified: commons/proper/commons-site/trunk/src/site/xdoc/patches.xml Modified: commons/proper/commons-site/trunk/src/site/xdoc/patches.xml URL: http://svn.apache.org/viewvc/commons/proper/commons-site/trunk/src/site/xdoc/patches.xml?rev=1412496&r1=1412495&r2=1412496&view=diff ============================================================================== --- commons/proper/commons-site/trunk/src/site/xdoc/patches.xml (original) +++ commons/proper/commons-site/trunk/src/site/xdoc/patches.xml Thu Nov 22 10:59:47 2012 @@ -104,7 +104,7 @@ </p> <p> 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/> + <source>svn diff <em>File</em> > <em>patchfile</em></source> </p> <p> The patch should be created from the project root, i.e. the directory which contains the src/ folder @@ -116,6 +116,12 @@ <p> Try to give your patch files meaningful names. This makes it easier for developers who need to apply a number of different patches. + Also the file should have an extension of .patch or .txt; files without extensions tend to be flagged + as binary by JIRA, which makes them awkward to use. + </p> + <p> + For example: + <source>svn diff <em>src/site/xdoc/patches.xml</em> > <em>COMMONSSITE-123.patch</em></source><br/> </p> </section> <section name="Submitting A Patch">