Difference between revisions of "Sending Patches VLC"
Jump to navigation
Jump to search
Pdherbemont (talk | contribs) (Initial Creation.) |
Pdherbemont (talk | contribs) |
||
| Line 7: | Line 7: | ||
== Check List == | == Check List == | ||
When you send a patch make sure that: | When you send a patch make sure that: | ||
| − | * it mostly complies to the [[Code | + | * it mostly complies to the [[Code Conventions]] |
* Make sure that it does not add more failure to `make check` | * Make sure that it does not add more failure to `make check` | ||
* Make sure that your patch is produced against the latest trunk. (Most of the time patch are against branches version such as 0.8.6 series which is not a development version. If this is a bugfix it may be backported, but should go in trunk first.). | * Make sure that your patch is produced against the latest trunk. (Most of the time patch are against branches version such as 0.8.6 series which is not a development version. If this is a bugfix it may be backported, but should go in trunk first.). | ||
Revision as of 13:25, 1 February 2008
Contents
How to produce a Patch
- Get the latest SVN trunk
- Make your change
- Produce a patch using:
svn diff
Check List
When you send a patch make sure that:
- it mostly complies to the Code Conventions
- Make sure that it does not add more failure to `make check`
- Make sure that your patch is produced against the latest trunk. (Most of the time patch are against branches version such as 0.8.6 series which is not a development version. If this is a bugfix it may be backported, but should go in trunk first.).
Sending it to the vlc-devel
Now you can send it to the vlc-devel. Please subscribe to it before sending your patch, or else it may not got through the list's spam filters. You will be able to unsuscribe later easily if needed.
Writing an appropriate description
- The patch email's subject should be prefixed by "[Patch]".
- You should include a description that will be the commit log message of your patch
- A more exhaustive explanation is also welcomed along with your patch.
Getting your patch merged
- Don't hesitate to ask for review if after a week there is no replies.
- If there are comments, please answer to those and eventually correct your patch if possible.
That should ensure that your patch gets merged.
