manPortico Estate - Cookbook: recipe #239, Some guidelines on handling patches

 
 

You are not allowed to post comments on this tracker with your current authentication level.

recipe #239: Some guidelines on handling patches

Submitter:  Sigurd Nes <sigurdne>
Submitted:  Sun 26 Sep 2010 03:21:18 PM UTC
   
 
Category:  None Importance:  3 - Normal
Status:  Approved Privacy:  Public
Assigned to:  None Open/Closed:  Open

Sun 26 Sep 2010 03:21:18 PM UTC, recipe preview:  

Patches


Our patch manager is hosted on savannah. This patch manager contains various patches which may be bug fixes, enhancements or malicious code. If they are open on savannah, they are untested and unsupported.

From time to time we may link to 3rd party patch sets. These patch sets have been tested by someone from the project, and appear to work ok, but again the project provides no official support or endorsement of such patch sets.

Getting the patch tool


If you need the patch command line tool, it is available with most GNU/Linux distributions, on Windows you can use MinGW, Cygwin or unxutils.

Submitting Patches


We appreciate all contributions to the project, but the contribution needs to be in a usable form. This document is designed to make it easy for us to work with the patches you provide to the project.

Clearly specify the purpose of the patch and the application name in the summary field, starting with ENH for enhancements or BUGFIX for bug fixes. In the "original submission", please include the following information. Only include one feature/fix in each patch submission.

Target Branch: 1.0, trunk etc
Format: diff
Affected file/s: file1.php, file2.php


DO NOT email patches, they will get lost. Also you don't need to worry about sending us an email after the patch has been submitted, savannah does that for you :)

Please upload your patch, don't paste it on savannah, it will mangle it, which makes it a lot of work for a developer to apply. Do not compress the patch, and if you are using Windows, please run dos2unix on the files before creating the patch.

Please don't point to websites. If we have to click off the patch manager to work on the patch, your patch is more likely to be ignored.

You may receive some comments regarding the patch, if there are problem these will have to be fixed before it is applied. Alternatively, the patch manager may modify the patch before commiting it to CVS. It is there call :)

Make your patches one subdirectory under the source tree you're working on. In short if you have the following two dirs /var/www/fmsystem.old and /var/www/fmsystem you should do the following commands:


$ cd /var/www
$ diff -aburN --exclude=.svn* fmsystem.old fmsystem > patchfile.diff


Then upload patchfile.diff to savannah as described above. We do it this way so that we can (regardless of where our source tree is) apply them with "patch -p1" from our fmsystem top directory. Like so:


$ cd /path/to/our/fmsystem
$ patch -p1 < ~/patchfile.diff


and we dont have to worry about three patches being based on /var/www/fmsystem, while two others not having a path at all and being individual filenames in fmsystem/phpgwapi/inc.

Please make sure you run a cvs update (and retest) before creating the patch to ensure it is upto date.

CVS diffs may be accepted, it depends on the mood of the person checking the patch. It is best to follow the steps above.

We thank you for all your submissions, and simply want to make sure your time isn't wasted and your patch ignored.

Sigurd Nes <sigurdne>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by sigurdne (Submitted the item)
  •  

    Follow 10 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-09-26 sigurdne StatusDraft Approved
    2010-09-26 sigurdne Recipe== Patches == Our... -13 chars
    2010-09-26 sigurdne Recipe== Patches == Our... -6 chars
    2010-09-26 sigurdne Recipe== Patches == Our... +12 chars
    2010-09-26 sigurdne Recipe== Patches == Our... +18 chars
    2010-09-26 sigurdne Recipe== Patches == Our... -4 chars
    2010-09-26 sigurdne Recipe== Patches == Our... +21 chars
    2010-09-26 sigurdne Recipe== Patches == Our... +84 chars
    2010-09-26 sigurdne RecipeOur patch manager is... +85 chars
    2010-09-26 sigurdne RecipeOur patch manager is... -4 chars

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code