Difference between revisions of "Development"
(→Submitting Patches: Rewrite to emphasize pull requests) |
|||
(14 intermediate revisions by 7 users not shown) | |||
Line 5: | Line 5: | ||
The GUI is built with [http://www.gtk.org GTK+] version 2.24. | The GUI is built with [http://www.gtk.org GTK+] version 2.24. | ||
− | + | We're supporting the [https://cmake.org CMake] build system. Understanding this complex system isn't generally necessary for fixing simple bugs, but more complex work will. Here's a [https://cmake.org/cmake-tutorial/ short CMake tutorial]. | |
− | We use [http://www.stack.nl/~dimitri/doxygen Doxygen] to document the sources, including the C API. Details about our use are on the [[Doxygen]] wiki page. The ''Help Manual'' and ''Concepts Guide'' are formatted using [http://www.docbook.org/ DocBook], an SGML markup language which enables us to | + | We use [http://www.stack.nl/~dimitri/doxygen Doxygen] to document the sources, including the C API. Details about our use are on the [[Doxygen]] wiki page. The ''Help Manual'' and ''Concepts Guide'' are formatted using [http://www.docbook.org/ DocBook], an SGML markup language which enables us to publish them in several formats. Note that these documents have a separate [https://github.com/Gnucash/gnucash-docs repository]. |
− | |||
− | |||
== Object Orientation == | == Object Orientation == | ||
Most of the code is written in an object-oriented style. Some of it uses [http://en.wikipedia.org/wiki/GObject GObject], some uses a home-grown GObject-like system called QofObject, and some just does what GObject does in straight C. Understanding how to use that is an important skill to developing for GnuCash. | Most of the code is written in an object-oriented style. Some of it uses [http://en.wikipedia.org/wiki/GObject GObject], some uses a home-grown GObject-like system called QofObject, and some just does what GObject does in straight C. Understanding how to use that is an important skill to developing for GnuCash. | ||
− | |||
+ | The next development cycle will include migrating those parts of Gnucash other than the GUI to [http://isocpp.org/ C++]; see our [[C++]] page for details and resources. We have updated the GUI to Gtk+-3 in the 2.7 series. | ||
+ | |||
+ | = What We Need Help On = | ||
+ | * Patches to fix [[Bugzilla|bugs]] are always a appreciated. It's an excellent way to get familiar with the code base and to introduce yourself to the team. | ||
+ | * If you prefer to dive in at the deep end, have a look at the development team's [[Roadmap|long-term plans]]. | ||
= Preliminaries = | = Preliminaries = | ||
Please subscribe to the [[Mailing Lists]] and introduce yourself to the development team. Some of the team also hang out on [[IRC]] so you can interact more directly. | Please subscribe to the [[Mailing Lists]] and introduce yourself to the development team. Some of the team also hang out on [[IRC]] so you can interact more directly. | ||
− | Read the two files [ | + | Read the two files [https://github.com/Gnucash/gnucash/blob/master/HACKING HACKING] and [https://github.com/Gnucash/gnucash/blob/master/README.git README.git] |
− | Look through the [ | + | Look through the [https://code.gnucash.org/docs/MASTER/ API documentation]. |
= Getting Sources and Building = | = Getting Sources and Building = | ||
Line 30: | Line 32: | ||
* Please try to develop according to [http://en.wikipedia.org/wiki/Test-driven_development Test Driven Development] principles, following our [[Testing]] guidelines. | * Please try to develop according to [http://en.wikipedia.org/wiki/Test-driven_development Test Driven Development] principles, following our [[Testing]] guidelines. | ||
* Please follow our [[CodingStandard| coding style]]. | * Please follow our [[CodingStandard| coding style]]. | ||
− | * If your code will contain '''textual output for the user''', have a look at [[Translation#Tips_for_Developers | Translation: Tips for Developers]]. | + | * If your code will contain '''textual output for the user''', have a look at [[Translation#Tips_for_Developers | Translation: Tips for Developers]] and configuration [[Settings Wording]]. |
* Be sure that <tt>make check</tt> passes before preparing your patch or pushing your commit. | * Be sure that <tt>make check</tt> passes before preparing your patch or pushing your commit. | ||
Line 44: | Line 46: | ||
Once you have your changes written and well tested—<tt>make check</tt> will run a bunch of tests—you'll want to submit it so that someone with commit privilege can add it to the official sources. There are two ways we accept code: | Once you have your changes written and well tested—<tt>make check</tt> will run a bunch of tests—you'll want to submit it so that someone with commit privilege can add it to the official sources. There are two ways we accept code: | ||
− | # [[Git#Pull_Requests|Github pull requests]]: This is the preferred method if the change is non-trivial | + | # [[Git#Pull_Requests|Github pull requests]]: This is the preferred method if the change is non-trivial. |
+ | #* If there is already a [[Bugzilla|bug report]] on the matter be sure to include a link to the bug in the pull request and comment on the bug with a link to the pull request. | ||
# Attach a patch to a [[Bugzilla|bug report]]: | # Attach a patch to a [[Bugzilla|bug report]]: | ||
:* [[Git#Patches|Make a patch]]. | :* [[Git#Patches|Make a patch]]. | ||
Line 57: | Line 60: | ||
* Most developers seem to have used [[Emacs]] as IDE. | * Most developers seem to have used [[Emacs]] as IDE. | ||
* Additionally there are some experiences with [[Eclipse]]. | * Additionally there are some experiences with [[Eclipse]]. | ||
+ | * Still to be documented: [[KDevelop]]. | ||
* Some are using [http://qt-project.org/wiki/Category:Tools::QtCreator QtCreator]. | * Some are using [http://qt-project.org/wiki/Category:Tools::QtCreator QtCreator]. | ||
Line 68: | Line 72: | ||
An [https://lists.gnucash.org/pipermail/gnucash-devel/2004-June/011547.html informative mail] from the archives. | An [https://lists.gnucash.org/pipermail/gnucash-devel/2004-June/011547.html informative mail] from the archives. | ||
− | [http://svn.gnucash.org/docs/ | + | ;GnuCash design and developer's reference: [http://svn.gnucash.org/docs/MAINT/ current bugfix] and [http://svn.gnucash.org/docs/MASTER/ future features] branch. |
[http://www.gnu.org/software/guile/manual/html_node/Programming-in-Scheme.html Scheme programming manual] | [http://www.gnu.org/software/guile/manual/html_node/Programming-in-Scheme.html Scheme programming manual] | ||
[[C API]] | [[C API]] | ||
+ | |||
+ | |||
+ | [[Category:Development]] |
Latest revision as of 18:50, 1 April 2018
Contents
Overview
GnuCash is written principally in C. A Guile interpreter is built in and parts of Gnucash--principally reports, but also parts of the user configuration, file import, and other small parts--are in Scheme. The C API is wrapped for Guile access with SWIG; Python wrappers can be created with a configure option.
The GUI is built with GTK+ version 2.24.
We're supporting the CMake build system. Understanding this complex system isn't generally necessary for fixing simple bugs, but more complex work will. Here's a short CMake tutorial.
We use Doxygen to document the sources, including the C API. Details about our use are on the Doxygen wiki page. The Help Manual and Concepts Guide are formatted using DocBook, an SGML markup language which enables us to publish them in several formats. Note that these documents have a separate repository.
Object Orientation
Most of the code is written in an object-oriented style. Some of it uses GObject, some uses a home-grown GObject-like system called QofObject, and some just does what GObject does in straight C. Understanding how to use that is an important skill to developing for GnuCash.
The next development cycle will include migrating those parts of Gnucash other than the GUI to C++; see our C++ page for details and resources. We have updated the GUI to Gtk+-3 in the 2.7 series.
What We Need Help On
- Patches to fix bugs are always a appreciated. It's an excellent way to get familiar with the code base and to introduce yourself to the team.
- If you prefer to dive in at the deep end, have a look at the development team's long-term plans.
Preliminaries
Please subscribe to the Mailing Lists and introduce yourself to the development team. Some of the team also hang out on IRC so you can interact more directly.
Read the two files HACKING and README.git
Look through the API documentation.
Getting Sources and Building
You can follow the git instructions for cloning the repository and preparing patches.
Build instructions for various platforms are described or linked at Building.
Coding Guidelines
- Please try to develop according to Test Driven Development principles, following our Testing guidelines.
- Please follow our coding style.
- If your code will contain textual output for the user, have a look at Translation: Tips for Developers and configuration Settings Wording.
- Be sure that make check passes before preparing your patch or pushing your commit.
Documentation
If your code adds or changes some functionality, do not forget the documentation.
- Make sure that global (i.e. not static) functions are documented with Doxygen-formatted comments.
- Try to keep the README files and that in src/doc up to date.
- Update the relevant sections in Help Manual and the Concept Guide.
Submitting Patches
Once you have your changes written and well tested—make check will run a bunch of tests—you'll want to submit it so that someone with commit privilege can add it to the official sources. There are two ways we accept code:
- Github pull requests: This is the preferred method if the change is non-trivial.
- If there is already a bug report on the matter be sure to include a link to the bug in the pull request and comment on the bug with a link to the pull request.
- Attach a patch to a bug report:
- Make a patch.
- If there's already a bug about it in Bugzilla, just attach the patch to the bug. Be sure to check the "patch" checkbox on the attachment form.
- If there isn't a bug already (be sure to search!), you'll need to create a new one to attach your patch to:
- Describe the problem or improvement that your patch addresses in the initial comment.
- Open the bug before you commit your changes so that you can include the bug number in the commit message.
Tools
Text editors / IDE:
- Most developers seem to have used Emacs as IDE.
- Additionally there are some experiences with Eclipse.
- Still to be documented: KDevelop.
- Some are using QtCreator.
For the GTK GUI:
Some may be interested in our experiments with CMake and Qt: Cutecash.
See also
An informative mail from the archives.
- GnuCash design and developer's reference
- current bugfix and future features branch.