Mercurial > hg > orthanc-book
changeset 627:53992cb7fa7f
c++03
author | Sebastien Jodogne <s.jodogne@gmail.com> |
---|---|
date | Sat, 06 Mar 2021 11:35:51 +0100 |
parents | 68c6c251e348 |
children | e6a2bcf729b4 |
files | Sphinx/source/developers/repositories.rst Sphinx/source/faq/licensing.rst |
diffstat | 2 files changed, 51 insertions(+), 28 deletions(-) [+] |
line wrap: on
line diff
--- a/Sphinx/source/developers/repositories.rst Thu Mar 04 10:55:12 2021 +0100 +++ b/Sphinx/source/developers/repositories.rst Sat Mar 06 11:35:51 2021 +0100 @@ -155,13 +155,58 @@ <https://www.mercurial-scm.org/wiki/CommunicatingChanges>`__: by submitting a patch, or by exchanging a bundle. -Importantly, before any contribution can be accepted into the Orthanc -repositories, its author must sign a :ref:`CLA <cla>`. This allows -both the University Hospital of Liège and the Osimis company to act as -the official guardians of the whole Orthanc ecosystem. + +Code quality +............ + +* Your code **must follow the C++03 standard** (C++11 is not accepted + for maximum cross-platform compatibility on older platforms). + +* The continuous integration servers of Osimis check that Orthanc + properly compiles on Ubuntu 16.04, on `Linux Standard Base systems + <https://refspecs.linuxfoundation.org/lsb.shtml>`__ using the `LSB + SDB 5.0.0 + <http://ftp.linuxfoundation.org/pub/lsb/bundles/released-5.0.0/sdk/>`__, + on FreeBSD, on Microsoft Visual Studio 2008 (32 bit), on Microsoft + Visual Studio 2015 (64 bit), and on Apple OS X 10.9 + "Mavericks". Submitted code might have to be adapted to compile on + these platforms. Architecture-dependant code should be located in + the `Orthanc::Toolbox + <https://hg.orthanc-server.com/orthanc/file/default/OrthancFramework/Sources/Toolbox.h>`__ + and `Orthanc::SystemToolbox + <https://hg.orthanc-server.com/orthanc/file/default/OrthancFramework/Sources/SystemToolbox.h>`__ + static classes. + +* Please stick to the :ref:`coding style <coding-style>` of Orthanc. -Also, make sure to read our :ref:`FAQ if submitting code -<submitting_code>`. +* Your individual contributions should be kept **as small as + possible**, and should be focused on one very specific issue or + feature. Large architectural changes are reserved for the core + development team of Orthanc, as we must follow our `long-term + roadmap + <https://hg.orthanc-server.com/orthanc/file/default/TODO>`__. + +* Unit testing is mandatory. Integration tests should be submitted to + the `dedicated repository + <https://hg.orthanc-server.com/orthanc-tests/file/default>`__. + +* All the contributions will be carefully reviewed. Some contributions + may be modified, yet even rejected. A rejection might for instance + occur if your contribution does not match the Orthanc roadmap, does + not meet our high-quality code standards, or breaks backward + compatibility. Please be sure that we warmly welcome and appreciate + your contributions, but be aware of the fact that we are quite + strict, and that the review process might take time. This is why the + recommended way of contributing to Orthanc is always by + :ref:`creating contributed plugins <creating-plugins>`. + +* Importantly, before any contribution can be accepted into the + Orthanc repositories, its author must sign a :ref:`CLA <cla>`. This + allows both the University Hospital of Liège and the Osimis company + to act as the official guardians of the whole Orthanc + ecosystem. Another advantage of creating plugins is that signing a + CLA is not needed. Make sure to carefully read our :ref:`FAQ about + the licensing of submitted code <submitting_code>`. .. _hg-patch:
--- a/Sphinx/source/faq/licensing.rst Thu Mar 04 10:55:12 2021 +0100 +++ b/Sphinx/source/faq/licensing.rst Sat Mar 06 11:35:51 2021 +0100 @@ -261,32 +261,10 @@ **Some words of warning:** -* Please stick to the :ref:`coding style <coding-style>` of Orthanc. - * It is your responsibility to make sure that you have the intellectual property over all the source code you commit into Orthanc. -* All the contributions will be carefully reviewed. Some contributions - may be modified, yet even rejected. A rejection might for instance - occur if your contribution does not match the Orthanc roadmap, does - not meet our high-quality code standards, or breaks backward - compatibility. Please be sure that we warmly welcome and appreciate - your contributions, but be aware of the fact that we are quite - strict, and that the review process might take time. This is why the - recommended way of contributing to Orthanc is always by - :ref:`creating contributed plugins <creating-plugins>`. - -* Your pull requests should be kept **as small as possible**, and - should be focused on one very specific issue or feature. Large - architectural changes are reserved for the core development team of - Orthanc, as we must follow our `long-term roadmap - <https://hg.orthanc-server.com/orthanc/file/default/TODO>`__. - -* Unit testing is mandatory. Integration tests should be submitted to - the `dedicated repository - <https://hg.orthanc-server.com/orthanc-tests/file/default>`__. - * In the case of a doubt wrt. a potential contribution, please discuss it on the `Orthanc Users <https://groups.google.com/forum/#!forum/orthanc-users>`__