| 1 |
24 |
jeremybenn |
|
| 2 |
|
|
Contributing to GDB
|
| 3 |
|
|
|
| 4 |
|
|
GDB is a collaborative project and one which wants to encourage new
|
| 5 |
|
|
development. You may wish to fix GDB bugs, improve testing, port GDB
|
| 6 |
|
|
to a new platform, update documentation, add new GDB features, and the
|
| 7 |
|
|
like. To help with this, there is a lot of documentation
|
| 8 |
|
|
available.. In addition to the user guide and internals manual
|
| 9 |
|
|
included in the GDB distribution, the GDB web pages also contain much
|
| 10 |
|
|
information.
|
| 11 |
|
|
|
| 12 |
|
|
You may also want to submit your change so that can be considered for
|
| 13 |
|
|
conclusion in a future version of GDB (see below). Regardless, we
|
| 14 |
|
|
encourage you to distribute the change yourself.
|
| 15 |
|
|
|
| 16 |
|
|
If you don't feel up to hacking GDB, there are still plenty of ways to
|
| 17 |
|
|
help! You can answer questions on the mailing lists, write
|
| 18 |
|
|
documentation, find bugs, create a GDB related website (contribute to
|
| 19 |
|
|
the official GDB web site), or create a GDB related software
|
| 20 |
|
|
package. We welcome all of the above and feel free to ask on the GDB
|
| 21 |
|
|
mailing lists if you are looking for feedback or for people to review
|
| 22 |
|
|
a work in progress.
|
| 23 |
|
|
|
| 24 |
|
|
Ref: http://www.gnu.org/software/gdb/
|
| 25 |
|
|
|
| 26 |
|
|
Finally, there are certain legal requirements and style issues which
|
| 27 |
|
|
all contributors need to be aware of.
|
| 28 |
|
|
|
| 29 |
|
|
o Coding Standards
|
| 30 |
|
|
|
| 31 |
|
|
All contributions must conform to the GNU Coding Standard.
|
| 32 |
|
|
Submissions which do not conform to the standards will be
|
| 33 |
|
|
returned with a request to reformat the changes.
|
| 34 |
|
|
|
| 35 |
|
|
GDB has certain additional coding requirements. Those
|
| 36 |
|
|
requirements are explained in the GDB internals documentation
|
| 37 |
|
|
in the gdb/doc directory.
|
| 38 |
|
|
|
| 39 |
|
|
Ref: http://www.gnu.org/prep/standards_toc.html
|
| 40 |
|
|
|
| 41 |
|
|
|
| 42 |
|
|
o Copyright Assignment
|
| 43 |
|
|
|
| 44 |
|
|
Before we can accept code contributions from you, we need a
|
| 45 |
|
|
copyright assignment form filled out and filed with the FSF.
|
| 46 |
|
|
|
| 47 |
|
|
See some documentation by the FSF for details and contact us
|
| 48 |
|
|
(either via the GDB mailing list or the GDB maintainer that is
|
| 49 |
|
|
taking care of your contributions) to obtain the relevant
|
| 50 |
|
|
forms.
|
| 51 |
|
|
|
| 52 |
|
|
Small changes can be accepted without a copyright assignment form on file.
|
| 53 |
|
|
|
| 54 |
|
|
Ref: http://www.gnu.org/prep/maintain.html#SEC6
|
| 55 |
|
|
|
| 56 |
|
|
|
| 57 |
|
|
o Submitting Patches
|
| 58 |
|
|
|
| 59 |
|
|
Every patch must have several pieces of information before we
|
| 60 |
|
|
can properly evaluate it.
|
| 61 |
|
|
|
| 62 |
|
|
A description of the bug and how your patch fixes this
|
| 63 |
|
|
bug. A reference to a testsuite failure is very helpful. For
|
| 64 |
|
|
new features a description of the feature and your
|
| 65 |
|
|
implementation.
|
| 66 |
|
|
|
| 67 |
|
|
A ChangeLog entry as plaintext (separate from the patch); see
|
| 68 |
|
|
the various ChangeLog files for format and content. Note that,
|
| 69 |
|
|
unlike some other projects, we do require ChangeLogs also for
|
| 70 |
|
|
documentation (i.e., .texi files).
|
| 71 |
|
|
|
| 72 |
|
|
The patch itself. If you are accessing the CVS repository use
|
| 73 |
|
|
"cvs update; cvs diff -cp"; else, use "diff -cp OLD NEW" or
|
| 74 |
|
|
"diff -up OLD NEW". If your version of diff does not support
|
| 75 |
|
|
these options, then get the latest version of GNU diff.
|
| 76 |
|
|
|
| 77 |
|
|
We accept patches as plain text (preferred for the compilers
|
| 78 |
|
|
themselves), MIME attachments (preferred for the web pages),
|
| 79 |
|
|
or as uuencoded gzipped text.
|
| 80 |
|
|
|
| 81 |
|
|
When you have all these pieces, bundle them up in a mail
|
| 82 |
|
|
message and send it to gdb-patches@sourceware.org. All
|
| 83 |
|
|
patches and related discussion should be sent to the
|
| 84 |
|
|
gdb-patches mailinglist. For further information on the GDB
|
| 85 |
|
|
CVS repository, see the Anonymous read-only CVS access and
|
| 86 |
|
|
Read-write CVS access page.
|
| 87 |
|
|
|
| 88 |
|
|
--
|
| 89 |
|
|
|
| 90 |
|
|
Supplemental information for GDB:
|
| 91 |
|
|
|
| 92 |
|
|
o Please try to run the relevant testsuite before and after
|
| 93 |
|
|
committing a patch
|
| 94 |
|
|
|
| 95 |
|
|
If the contributor doesn't do it then the maintainer will. A
|
| 96 |
|
|
contributor might include before/after test results in their
|
| 97 |
|
|
contribution.
|
| 98 |
|
|
|
| 99 |
|
|
|
| 100 |
|
|
o For bug fixes, please try to include a way of
|
| 101 |
|
|
demonstrating that the patch actually fixes something.
|
| 102 |
|
|
|
| 103 |
|
|
The best way of doing this is to ensure that the
|
| 104 |
|
|
testsuite contains one or more test cases that
|
| 105 |
|
|
fail without the fix but pass with the fix.
|
| 106 |
|
|
|
| 107 |
|
|
People are encouraged to submit patches that extend
|
| 108 |
|
|
the testsuite.
|
| 109 |
|
|
|
| 110 |
|
|
|
| 111 |
|
|
o Please read your patch before submitting it.
|
| 112 |
|
|
|
| 113 |
|
|
A patch containing several unrelated changes or
|
| 114 |
|
|
arbitrary reformats will be returned with a request
|
| 115 |
|
|
to re-formatting / split it.
|
| 116 |
|
|
|
| 117 |
|
|
|
| 118 |
|
|
o If ``gdb/configure.in'' is modified then you don't
|
| 119 |
|
|
need to include patches to the regenerated file
|
| 120 |
|
|
``configure''.
|
| 121 |
|
|
|
| 122 |
|
|
The maintainer will re-generate those files
|
| 123 |
|
|
using autoconf (2.13 as of 2000-02-29).
|
| 124 |
|
|
|
| 125 |
|
|
|
| 126 |
|
|
o If ``gdb/gdbarch.sh'' is modified, you don't
|
| 127 |
|
|
need to include patches to the generated files
|
| 128 |
|
|
``gdbarch.h'' and ``gdbarch.c''.
|
| 129 |
|
|
|
| 130 |
|
|
See ``gdb/configure.in'' above.
|
| 131 |
|
|
|
| 132 |
|
|
|
| 133 |
|
|
o When submitting a patch that fixes a bug
|
| 134 |
|
|
in GDB's bug database a brief reference
|
| 135 |
|
|
to the bug can be included in the ChangeLog
|
| 136 |
|
|
vis
|
| 137 |
|
|
|
| 138 |
|
|
* CONTRIBUTE: Mention PR convention.
|
| 139 |
|
|
Fix PR gdb/4705.
|
| 140 |
|
|
|
| 141 |
|
|
The text ``PR gdb/4705'' should also be included
|
| 142 |
|
|
in the CVS commit message. That causes the
|
| 143 |
|
|
patch to automatically be archived with the PR.
|