| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | HOW TO CONTRIBUTE TO PATCHES OpenSSL | 
					
						
							|  |  |  | ------------------------------------ | 
					
						
							| 
									
										
										
										
											2015-11-01 21:38:42 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  | (Please visit https://www.openssl.org/community/getting-started.html for | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | other ideas about how to contribute.) | 
					
						
							| 
									
										
										
										
											2015-11-01 21:38:42 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | Development is coordinated on the openssl-dev mailing list (see the | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  | above link or https://mta.openssl.org for information on subscribing). | 
					
						
							| 
									
										
										
										
											2015-11-01 21:38:42 +08:00
										 |  |  | If you are unsure as to whether a feature will be useful for the general | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | OpenSSL community you might want to discuss it on the openssl-dev mailing | 
					
						
							|  |  |  | list first.  Someone may be already working on the same thing or there | 
					
						
							|  |  |  | may be a good reason as to why that feature isn't implemented. | 
					
						
							| 
									
										
										
										
											2015-11-01 21:38:42 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | The best way to submit a patch is to make a pull request on GitHub. | 
					
						
							|  |  |  | (It is not necessary to send mail to rt@openssl.org to open a ticket!) | 
					
						
							|  |  |  | If you think the patch could use feedback from the community, please | 
					
						
							|  |  |  | start a thread on openssl-dev. | 
					
						
							| 
									
										
										
										
											2015-11-01 21:38:42 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  | You can also submit patches by sending it as mail to rt@openssl.org. | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | Please include the word "PATCH" and an explanation of what the patch | 
					
						
							|  |  |  | does in the subject line.  If you do this, our preferred format is "git | 
					
						
							|  |  |  | format-patch" output. For example to provide a patch file containing the | 
					
						
							|  |  |  | last commit in your local git repository use the following command: | 
					
						
							| 
									
										
										
										
											2015-11-01 22:41:29 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  |     % git format-patch --stdout HEAD^ >mydiffs.patch | 
					
						
							| 
									
										
										
										
											2015-11-01 22:41:29 +08:00
										 |  |  | 
 | 
					
						
							|  |  |  | Another method of creating an acceptable patch file without using git is as | 
					
						
							|  |  |  | follows: | 
					
						
							| 
									
										
										
										
											2015-11-01 21:38:42 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  |     % cd openssl-work | 
					
						
							|  |  |  |     ...make your changes... | 
					
						
							|  |  |  |     % ./Configure dist; make clean | 
					
						
							|  |  |  |     % cd .. | 
					
						
							|  |  |  |     % diff -ur openssl-orig openssl-work >mydiffs.patch | 
					
						
							|  |  |  | 
 | 
					
						
							|  |  |  | Note that pull requests are generally easier for the team, and community, to | 
					
						
							|  |  |  | work with.  Pull requests benefit from all of the standard GitHub features, | 
					
						
							|  |  |  | including code review tools, simpler integration, and CI build support. | 
					
						
							|  |  |  | 
 | 
					
						
							|  |  |  | No matter how a patch is submitted, the following items will help make | 
					
						
							|  |  |  | the acceptance and review process faster: | 
					
						
							|  |  |  | 
 | 
					
						
							|  |  |  |     1. Anything other than trivial contributions will require a contributor | 
					
						
							|  |  |  |     licensing agreement, giving us permission to use your code. See | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  |     https://www.openssl.org/policies/cla.html for details. | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | 
 | 
					
						
							|  |  |  |     2.  All source files should start with the following text (with | 
					
						
							|  |  |  |     appropriate comment characters at the start of each line and the | 
					
						
							|  |  |  |     year(s) updated): | 
					
						
							|  |  |  | 
 | 
					
						
							|  |  |  |         Copyright 20xx-20yy The OpenSSL Project Authors. All Rights Reserved. | 
					
						
							|  |  |  | 
 | 
					
						
							|  |  |  |         Licensed under the OpenSSL license (the "License").  You may not use | 
					
						
							|  |  |  |         this file except in compliance with the License.  You can obtain a copy | 
					
						
							|  |  |  |         in the file LICENSE in the source distribution or at | 
					
						
							|  |  |  |         https://www.openssl.org/source/license.html | 
					
						
							|  |  |  | 
 | 
					
						
							|  |  |  |     3.  Patches should be as current as possible.  When using GitHub, please | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  |     expect to have to rebase and update often. Note that we do not accept merge | 
					
						
							|  |  |  |     commits. You will be asked to remove them before a patch is considered | 
					
						
							|  |  |  |     acceptable. | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  |     4.  Patches should follow our coding style (see | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  |     https://www.openssl.org/policies/codingstyle.html) and compile without | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  |     warnings. Where gcc or clang is availble you should use the | 
					
						
							|  |  |  |     --strict-warnings Configure option.  OpenSSL compiles on many varied | 
					
						
							|  |  |  |     platforms: try to ensure you only use portable features. | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  | 
 | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  |     5.  When at all possible, patches should include tests. These can either be | 
					
						
							| 
									
										
										
										
											2016-05-12 04:46:44 +08:00
										 |  |  |     added to an existing test, or completely new.  Please see test/README | 
					
						
							|  |  |  |     for information on the test framework. | 
					
						
							| 
									
										
										
										
											2016-06-04 00:12:08 +08:00
										 |  |  | 
 | 
					
						
							|  |  |  |     6.  New features or changed functionality must include documentation. Please | 
					
						
							|  |  |  |     look at the "pod" files in doc/apps, doc/crypto and doc/ssl for examples of | 
					
						
							|  |  |  |     our style. |