[libcamera-ci,0/4] Catch missing license information in CI
mbox series

Message ID 20260405192211.1210985-1-laurent.pinchart@ideasonboard.com
Headers show
Series
  • Catch missing license information in CI
Related show

Message

Laurent Pinchart April 5, 2026, 7:22 p.m. UTC 
Hello,

libcamera uses SPDX headers to convey license information, and the REUSE
specification and tool to ensure all files have a license. As the rule
is not enforced by tools, files with missing license information
regularly slip in.

This series adds a CI job to catch missing license information. It
should be applied after merging "[PATCH 00/11] libcamera: Fix missing
license information" ([1]) in libcamera to fix existing license
identifiers issues.

Patch 1/4 is a small drive by cleanup that drops a now unneeded meson
option. Patches 2/4 and 3/4 move some jobs from Debian 12 to Debian 13,
and then installs the packages those jobs depend on in the Debian 13
container only. Patch 4/4 finally adds the new license check job with
the REUSE tool.

[1] https://patchwork.libcamera.org/project/libcamera/list/?series=5866

Laurent Pinchart (4):
  Drop manual C++20 selection
  Move jobs to Debian 13
  Install documentation and lint packages in Debian 13 container only
  Add lint job to check licenses

 .gitlab-ci/lint-reuse.sh          | 26 ++++++++++++++++++++
 .gitlab-ci/meson/libc++.native    |  2 ++
 .gitlab-ci/meson/libstdc++.native |  2 ++
 .gitlab-ci/setup-container.sh     | 28 ++++++++++------------
 gitlab-ci.yml                     | 40 ++++++++++++++++++++-----------
 5 files changed, 68 insertions(+), 30 deletions(-)
 create mode 100755 .gitlab-ci/lint-reuse.sh


base-commit: 76dc8c2d642c562f372b6147e94e904f119a66f1

Comments

Laurent Pinchart April 5, 2026, 7:40 p.m. UTC | #1 
On Sun, Apr 05, 2026 at 10:22:04PM +0300, Laurent Pinchart wrote:
> Hello,
> 
> libcamera uses SPDX headers to convey license information, and the REUSE
> specification and tool to ensure all files have a license. As the rule
> is not enforced by tools, files with missing license information
> regularly slip in.
> 
> This series adds a CI job to catch missing license information. It
> should be applied after merging "[PATCH 00/11] libcamera: Fix missing
> license information" ([1]) in libcamera to fix existing license
> identifiers issues.
> 
> Patch 1/4 is a small drive by cleanup that drops a now unneeded meson
> option. Patches 2/4 and 3/4 move some jobs from Debian 12 to Debian 13,
> and then installs the packages those jobs depend on in the Debian 13
> container only. Patch 4/4 finally adds the new license check job with
> the REUSE tool.
> 
> [1] https://patchwork.libcamera.org/project/libcamera/list/?series=5866

Here's the log of the license lint job with [1] applied:

https://gitlab.freedesktop.org/pinchartl/libcamera/-/jobs/96843101

> Laurent Pinchart (4):
>   Drop manual C++20 selection
>   Move jobs to Debian 13
>   Install documentation and lint packages in Debian 13 container only
>   Add lint job to check licenses
> 
>  .gitlab-ci/lint-reuse.sh          | 26 ++++++++++++++++++++
>  .gitlab-ci/meson/libc++.native    |  2 ++
>  .gitlab-ci/meson/libstdc++.native |  2 ++
>  .gitlab-ci/setup-container.sh     | 28 ++++++++++------------
>  gitlab-ci.yml                     | 40 ++++++++++++++++++++-----------
>  5 files changed, 68 insertions(+), 30 deletions(-)
>  create mode 100755 .gitlab-ci/lint-reuse.sh
> 
> 
> base-commit: 76dc8c2d642c562f372b6147e94e904f119a66f1