| Contributing to GDB | |
| GDB is a collaborative project and one which wants to encourage new | |
| development. You may wish to fix GDB bugs, improve testing, port GDB | |
| to a new platform, update documentation, add new GDB features, and the | |
| like. To help with this, there is a lot of documentation | |
| available.. In addition to the user guide and internals manual | |
| included in the GDB distribution, the GDB web pages also contain much | |
| information. | |
| You may also want to submit your change so that can be considered for | |
| conclusion in a future version of GDB (see below). Regardless, we | |
| encourage you to distribute the change yourself. | |
| If you don't feel up to hacking GDB, there are still plenty of ways to | |
| help! You can answer questions on the mailing lists, write | |
| documentation, find bugs, create a GDB related website (contribute to | |
| the official GDB web site), or create a GDB related software | |
| package. We welcome all of the above and feel free to ask on the GDB | |
| mailing lists if you are looking for feedback or for people to review | |
| a work in progress. | |
| Ref: http://sourceware.cygnus.com/gdb | |
| Finally, there are certain legal requirements and style issues which | |
| all contributors need to be aware of. | |
| o Coding Standards | |
| All contributions must conform to the GNU Coding Standard. | |
| http://www.gnu.ai.mit.edu/prep/standards_toc.html | |
| Submissions which do not conform to the standards will be | |
| returned with a request to reformat the changes. | |
| For GDB, that standard is more tightly defined. GDB's | |
| coding standard is determined by the output of | |
| gnu-indent. | |
| This situation came about because, by the start of '99, | |
| GDB's coding style was so bad an inconsistent that it was | |
| decided to restart things from scratch. | |
| o Copyright Assignment | |
| There are certain legal requirements | |
| Before we can accept code contributions from you, we need a | |
| copyright assignment form filled out. | |
| If you've developed some addition or patch to GDB that you | |
| would like to contribute, you should fill out a copyright | |
| assignment form and send it in to the FSF. We are unable to | |
| use code from you until this is on-file at the FSF, so get | |
| that paperwork in! This form covers one batch of changes. | |
| Ref: http://gcc.gnu.org/fsf-forms/assignment-instructions.html | |
| If you think you're going to be doing continuing work on GDB, it | |
| would be easier to use a different form, which arranges to | |
| assign the copyright for all your future changes to GDB. It is | |
| called assign.future. Please note that if you switch | |
| employers, the new employer will need to fill out the | |
| disclaim.future form; there is no need to fill out the | |
| assign.future form again. | |
| Ref: http://gcc.gnu.org/fsf-forms/assign.future | |
| Ref: http://gcc.gnu.org/fsf-forms/disclaim.future | |
| There are several other forms you can fill out for different | |
| circumstances (e.g. to contribute an entirely new program, to | |
| contribute significant changes to a manual, etc.) | |
| Ref: http://gcc.gnu.org/fsf-forms/copyrights.html | |
| Small changes can be accepted without a copyright assignment | |
| form on file. | |
| This is pretty confusing! If you are unsure of what is | |
| necessary, just ask the GDB mailing list and we'll figure out | |
| what is best for you. | |
| Note: Many of these forms have a place for "name of | |
| program". Insert the name of one program in that place -- in | |
| this case, "GDB". | |
| o Submitting Patches | |
| Every patch must have several pieces of information before we | |
| can properly evaluate it. | |
| A description of the bug and how your patch fixes this | |
| bug. A reference to a testsuite failure is very helpful. For | |
| new features a description of the feature and your | |
| implementation. | |
| A ChangeLog entry as plaintext (separate from the patch); see | |
| the various ChangeLog files for format and content. Note that, | |
| unlike some other projects, we do require ChangeLogs also for | |
| documentation (i.e., .texi files). | |
| The patch itself. If you are accessing the CVS repository at: | |
| Cygnus, use "cvs update; cvs diff -c3p"; else, use "diff -c3p | |
| OLD NEW" or "diff -up OLD NEW". If your version of diff does | |
| not support these options, then get the latest version of GNU | |
| diff. | |
| We accept patches as plain text (preferred for the compilers | |
| themselves), MIME attachments (preferred for the web pages), | |
| or as uuencoded gzipped text. | |
| When you have all these pieces, bundle them up in a mail | |
| message and send it to gdb-patches@sourceware.cygnus.com. All | |
| patches and related discussion should be sent to the | |
| gdb-patches mailinglist. For further information on the GDB | |
| CVS repository, see the Anonymous read-only CVS access and | |
| Read-write CVS access page. | |
| -- | |
| Supplemental information for GDB: | |
| o Please try to run the relevant testsuite before and after | |
| committing a patch | |
| If the contributor doesn't do it then the maintainer will. A | |
| contributor might include before/after test results in their | |
| contribution. | |
| o For bug fixes, please try to include a way of | |
| demonstrating that the patch actually fixes something. | |
| The best way of doing this is to ensure that the | |
| testsuite contains one or more test cases that | |
| fail without the fix but pass with the fix. | |
| People are encouraged to submit patches that extend | |
| the testsuite. | |
| o Please read your patch before submitting it. | |
| A patch containing several unrelated changes or | |
| arbitrary reformats will be returned with a request | |
| to re-formatting / split it. | |
| o If ``gdb/configure.in'' is modified then you don't | |
| need to include patches to the regenerated file | |
| ``configure''. | |
| The maintainer will re-generate those files | |
| using autoconf (2.13 as of 2000-02-29). | |
| o If ``gdb/gdbarch.sh'' is modified, you don't | |
| need to include patches to the generated files | |
| ``gdbarch.h'' and ``gdbarch.c''. | |
| See ``gdb/configure.in'' above. | |
| o When submitting a patch that fixes a bug | |
| in GDB's bug database a brief reference | |
| to the bug can be included in the ChangeLog | |
| vis | |
| * CONTRIBUTE: Mention PR convention. | |
| Fix PR gdb/4705. | |
| The text ``PR gdb/4705'' should also be included | |
| in the CVS commit message. That causes the | |
| patch to automatically be archived with the PR. |