[libcamera-ci] Enable clang build on Debian trixie
diff mbox series

Message ID 20241215212254.19987-1-laurent.pinchart@ideasonboard.com
State New
Headers show
Series
  • [libcamera-ci] Enable clang build on Debian trixie
Related show

Commit Message

Laurent Pinchart Dec. 15, 2024, 9:22 p.m. UTC
Debian stable ships a relatively old clang version. Add a build test
with the most recent available clang version on trixie.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
Successful CI run for the libcamera master branch:
https://gitlab.freedesktop.org/pinchartl/libcamera/-/pipelines/1330345
---
 gitlab-ci.yml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Jacopo Mondi Dec. 16, 2024, 7:38 a.m. UTC | #1
Hi Laurent

On Sun, Dec 15, 2024 at 11:22:54PM +0200, Laurent Pinchart wrote:
> Debian stable ships a relatively old clang version. Add a build test
> with the most recent available clang version on trixie.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> Successful CI run for the libcamera master branch:
> https://gitlab.freedesktop.org/pinchartl/libcamera/-/pipelines/1330345
> ---
>  gitlab-ci.yml | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/gitlab-ci.yml b/gitlab-ci.yml
> index c78235ec1e52d571..06ab27dc0af0d9df 100644
> --- a/gitlab-ci.yml
> +++ b/gitlab-ci.yml
> @@ -224,6 +224,13 @@ build-full:debian:13:
>        - ARCH: amd64
>          CC: gcc-14
>          CXX: g++-14
> +      - ARCH: amd64
> +        CC: clang
> +        CXX: clang++
> +        MESON_OPTIONS: >-
> +          ${MESON_ALL_OPTIONS}
> +          ${MESON_CLANG_OPTIONS}
> +          -D qcam=disabled

Just wondering why we don't need these options for gcc, but mostly for
my education.

Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>

Thanks
  j

>
>  # Build each commit in the branch individually to detect compilation breakages.
>  build-history:
> --
> Regards,
>
> Laurent Pinchart
>
Laurent Pinchart Dec. 16, 2024, 9:50 a.m. UTC | #2
On Mon, Dec 16, 2024 at 08:38:44AM +0100, Jacopo Mondi wrote:
> On Sun, Dec 15, 2024 at 11:22:54PM +0200, Laurent Pinchart wrote:
> > Debian stable ships a relatively old clang version. Add a build test
> > with the most recent available clang version on trixie.
> >
> > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > ---
> > Successful CI run for the libcamera master branch:
> > https://gitlab.freedesktop.org/pinchartl/libcamera/-/pipelines/1330345
> > ---
> >  gitlab-ci.yml | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/gitlab-ci.yml b/gitlab-ci.yml
> > index c78235ec1e52d571..06ab27dc0af0d9df 100644
> > --- a/gitlab-ci.yml
> > +++ b/gitlab-ci.yml
> > @@ -224,6 +224,13 @@ build-full:debian:13:
> >        - ARCH: amd64
> >          CC: gcc-14
> >          CXX: g++-14
> > +      - ARCH: amd64
> > +        CC: clang
> > +        CXX: clang++
> > +        MESON_OPTIONS: >-
> > +          ${MESON_ALL_OPTIONS}
> > +          ${MESON_CLANG_OPTIONS}
> > +          -D qcam=disabled
> 
> Just wondering why we don't need these options for gcc, but mostly for
> my education.

git log and git blame have the answer :-)

> Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
> 
> >  # Build each commit in the branch individually to detect compilation breakages.
> >  build-history:

Patch
diff mbox series

diff --git a/gitlab-ci.yml b/gitlab-ci.yml
index c78235ec1e52d571..06ab27dc0af0d9df 100644
--- a/gitlab-ci.yml
+++ b/gitlab-ci.yml
@@ -224,6 +224,13 @@  build-full:debian:13:
       - ARCH: amd64
         CC: gcc-14
         CXX: g++-14
+      - ARCH: amd64
+        CC: clang
+        CXX: clang++
+        MESON_OPTIONS: >-
+          ${MESON_ALL_OPTIONS}
+          ${MESON_CLANG_OPTIONS}
+          -D qcam=disabled
 
 # Build each commit in the branch individually to detect compilation breakages.
 build-history: