Code with any of the warnings/errors mentioned in the policy, including strings externalization and API errors will not be accepted.

To minimize the change, do not re-format the source code you edited (except changed lines). Do not fix any warnings in the code you are not changing

If you really want to do formatting or styling (such as converting to Java 8) - create another bug for that and submit a gerrit (it is good to fix warnings but it would clutter the patch, you want to solve one problem at a time).

To speed up process of applying your changes you should create one or more junit tests as well and include it in your change

Gerrit is now used by the CDT project for contributions, follow the steps here to submit your changes to Gerrit. Changes will not be accepted via patches in Bugzilla.

Make sure your commit message references the bugzilla bug, e.g:

Bug 12345 fixed that, added this

A comment should be added to the bugzilla automatically once you post the change to Gerrit. If not please a new comment containing a link to the Gerrit so that people watching the bug know the a change has been posted for review, for example https://git.eclipse.org/r/#/c/8327/

Check size of the contribution, if it adds more than 1,000 lines see copyright section below

Make sure bug report has a clear reproducible scenario, if not add one

Normally committers are watching new Bugzilla/Gerrit activity and somebody would look at your contribution in a few days

If it has not received attention in a week or so, some nagging can help. Send email to mailto:cdt-dev@eclipse.org asking committers to look at the contribution. Continue sending e-mails until somebody would give up and look :)

One of the fundamental rules that Eclipse follows is the ability to trace back who contributed what code and that the person who contributed it has ownership of the code, or has permission from their employer to contribute the code (since employers tend to own everything you write). To help keep this IP integrity going, please ensure your contributions are "clean".

If you copy ANY code or images from somewhere else please clearly state it

If you copy GPL code we cannot take it

If you copy EPL code, preserve the original copyright and contributors

If your changes add more than 1,000 lines, the patch has to go through IP review process, unless it can be applied by the committer from the same company as you are. Nobody wants to do it so please avoid it. Try to fix one bug at a time.

The 1,000 lines include code, comments and even whitespace.

Signing your Eclipse Contributor Agreement (ECA)

The ECA is a short document that essentially asks the questions required for each contributions. It is mandatory to electronically sign a ECA before any contributions can be accepted. To sign your ECA:

Obtain an Eclipse Foundation userid. Anyone who currently uses our Bugzilla or Gerrit systems already has one of those. If they don’t, they need to register.

Login into the projects portal. On the right side of the page, in the Status section, click “Eclipse Contributor Agreement” then follow the instructions.

If you use Gerrit you might have to wait about an hour until ECA is available in Gerrit.