[PATCH v2 2/2] docs: Note new requirement to include a SPDX line

Stephen Finucane stephen at that.guru
Tue Sep 18 03:17:00 AEST 2018


Add some wording around the requirement to include this line instead of
the license header. Also note the requirement that all code be
GPLv2-licensed and add a CONTRIBUTING document, which GitHub likes.

Signed-off-by: Stephen Finucane <stephen at that.guru>
Cc: Daniel Axtens <dja at axtens.net>
---
 CONTRIBUTING.rst                  |  6 ++++++
 docs/development/contributing.rst | 20 +++++++++++++++++---
 2 files changed, 23 insertions(+), 3 deletions(-)
 create mode 100644 CONTRIBUTING.rst

diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst
new file mode 100644
index 00000000..131e2dcb
--- /dev/null
+++ b/CONTRIBUTING.rst
@@ -0,0 +1,6 @@
+Contributing
+============
+
+For guidelines on contributing, refer to the `contributors documentation`__.
+
+__ https://patchwork.readthedocs.io/en/latest/development/contributing/
diff --git a/docs/development/contributing.rst b/docs/development/contributing.rst
index 7e2a72cf..bada3938 100644
--- a/docs/development/contributing.rst
+++ b/docs/development/contributing.rst
@@ -4,13 +4,25 @@ Contributing
 Coding Standards
 ----------------
 
-**Follow PEP8**. All code is currently PEP8 compliant and it should stay this
-way.
+**Follow PEP8**. All code is currently `PEP 8`_ compliant and it should stay
+this way.
+
+All code must be GPLv2 licensed and must have a `SPDX License Identifier`_
+stating this. A copyright line should be included on new files and may be added
+for significant changes to existing files.
+
+.. code-block:: python
+
+   # Patchwork - automated patch tracking system
+   # Copyright (C) 2000 Jane Doe <jane.doe at example.com>
+   # Copyright (C) 2001 Joe Bloggs <joebloggs at example.com>
+   #
+   # SPDX-License-Identifier: GPL-2.0
 
 Changes that fix semantic issues will be generally be happily received, but
 please keep such changes separate from functional changes.
 
-`pep8` targets are provided via tox. Refer to the :ref:`testing` section
+``pep8`` targets are provided via tox. Refer to the :ref:`testing` section
 below for more information on usage of this tool.
 
 .. _testing:
@@ -148,6 +160,8 @@ announcements.
 Further information about the Patchwork mailing list is available can be found on
 `lists.ozlabs.org`_.
 
+.. _PEP 8: https://pep8.org/
+.. _SPDX License Identifier: https://spdx.org/using-spdx-license-identifier
 .. _tox: https://tox.readthedocs.io/en/latest/
 .. _reno: https://docs.openstack.org/developer/reno/
 .. _QEMU guidelines: http://wiki.qemu.org/Contribute/SubmitAPatch
-- 
2.17.1



More information about the Patchwork mailing list