Skip to main content

Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.

Jump to: navigation, search

Difference between revisions of "SWT/Devel"

< SWT
(Style)
 
(16 intermediate revisions by 2 users not shown)
Line 1: Line 1:
 
This page is intended to document the development of SWT itself. E.g compiling native bindings etc.
 
This page is intended to document the development of SWT itself. E.g compiling native bindings etc.
  
= SWT Platforms =
+
= Platform specific documentation =
SWT Development varies a lot depending on which platform you're developing on. Please see the individual sections.
+
SWT development varies a lot depending on which platform you're developing on.
  
====  SWT on GTK development ====
+
*[[SWT/Devel/Gtk | GTK development]]
Please see [[SWT/Devel/Gtk | GTK development.]]
+
 
+
==== SWT on Win32 development ====
+
  ... to be added some day.
+
 
+
==== SWT on Cocoa development ====
+
  ... to be added some day.
+
  
 
= Workflow =
 
= Workflow =
  
For workflow suggestions/simplification, please see the [[SWT/Devel/Workflow | SWT workflow page.]]
+
For workflow suggestions/simplification, as well as Bugzilla and Gerrit information, please see the [[SWT/Devel/Workflow | SWT workflow page.]]
== Using Gerrit ==
+
Gerrit is used to review your patches. For further information, please see [[SWT/Devel/Gerrit | gerrit.]]
+
 
+
== Backporting Patches ==
+
You have several options.
+
1) On the gerrit review of your patch, click on 'cherry pick' and start typing the name of your branch (e.g R4_5_maintenance)
+
2) Check out the remote repository locally, manually cherry pick the commit from your git history, then do either one of:
+
 
+
3a) '''Push directly into branch''' 
+
+
If you have committer rights and you would like to push your patch directly into a backport branch, execute from command line as following:
+
  git push review HEAD:R4_5_maintenance
+
 
+
3b) '''Create a review for backport'''
+
 
+
If you would like to submit an extra gerrit review before merging into a backport branch, execute as following:
+
  git push review HEAD:ref/for/R4_5_maintenance
+
(Note, the 'ref/for/' is a branch that gerrit monitors to create new patch reviews.)
+
 
+
== Mylyn ==
+
 
+
Nice Video tutorials:
+
* Mylyn overview: [https://www.youtube.com/watch?v=2wp32Ow_OmE Youtube : Mylyn Overview]
+
* Mylyn, doing code reviews on gerrit [https://youtu.be/gq5w6HAr_CE?t=2589 Youtube : Code review inside Eclipse with gerrit]
+
  Note, you need to install the 'Gerrit' connector inside mylyn.
+
  
= General Advice =
+
= Style =
==== Read Function Documentation ====
+
Always read documentation of every function that you are adding to the code base (esp. Gtk functions). 
+
  
This includes the situation where you copy & paste code because sometimes methods get deprecated and you end up copy & pasting deprecated methods. Copy & pasting code without knowing what it does exactly is dangerous.
+
For coding style/git practices, please see [[SWT/Devel/Style | SWT style guidelines.]]

Latest revision as of 13:32, 22 August 2018

This page is intended to document the development of SWT itself. E.g compiling native bindings etc.

Platform specific documentation

SWT development varies a lot depending on which platform you're developing on.

Workflow

For workflow suggestions/simplification, as well as Bugzilla and Gerrit information, please see the SWT workflow page.

Style

For coding style/git practices, please see SWT style guidelines.

Back to the top