[libcamera-ci,RFC,v2,2/4] Add `libevent` runtime libraries to the container
diff mbox series

Message ID 20241216172827.1055088-2-barnabas.pocze@ideasonboard.com
State Superseded
Headers show
Series
  • [libcamera-ci,RFC,v2,1/4] Enable `UDMABUF` in the kernel
Related show

Commit Message

Barnabás Pőcze Dec. 16, 2024, 5:28 p.m. UTC
`libevent` is needed for some components, specifically `cam`
and `lc-compliance`. Installing `libevent-dev` would be more
convenient, but it is not doable at the moment because that
package is not multiarch compatible. So only install the
necessary runtime libraries.

Build jobs already install `libevent-dev` when necessary before
building libcamera, but this change enables the aforementioned
components to be usable in different jobs as well.

Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
---
 .gitlab-ci/setup-container.sh | 3 +++
 gitlab-ci.yml                 | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

--
2.47.1

Comments

Laurent Pinchart Dec. 16, 2024, 11:23 p.m. UTC | #1
Hi Barnabás,

Thank you for the patch.

On Mon, Dec 16, 2024 at 06:28:25PM +0100, Barnabás Pőcze wrote:
> `libevent` is needed for some components, specifically `cam`
> and `lc-compliance`. Installing `libevent-dev` would be more
> convenient, but it is not doable at the moment because that
> package is not multiarch compatible. So only install the
> necessary runtime libraries.
> 
> Build jobs already install `libevent-dev` when necessary before
> building libcamera, but this change enables the aforementioned
> components to be usable in different jobs as well.
> 
> Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  .gitlab-ci/setup-container.sh | 3 +++
>  gitlab-ci.yml                 | 2 +-
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/.gitlab-ci/setup-container.sh b/.gitlab-ci/setup-container.sh
> index d2909c7..0658368 100755
> --- a/.gitlab-ci/setup-container.sh
> +++ b/.gitlab-ci/setup-container.sh
> @@ -103,6 +103,9 @@ case $FDO_DISTRIBUTION_VERSION in
>  'bookworm')
>  	# libclang-rt-dev for the clang ASan runtime.
>  	PKGS_LIBCAMERA_RUNTIME_MULTIARCH+=( libclang-rt-dev )
> +	# For cam and lc-compliance
> +	# libevent-dev cannot be used here, see build-libcamera-common.sh
> +	PKGS_LIBCAMERA_RUNTIME_MULTIARCH+=( libevent-2.1-7 libevent-pthreads-2.1-7 )
>  	;;
>  'trixie')
>  	# gcc 13 to expand compilation testing coverage.
> diff --git a/gitlab-ci.yml b/gitlab-ci.yml
> index 6d26680..f679d6e 100644
> --- a/gitlab-ci.yml
> +++ b/gitlab-ci.yml
> @@ -65,7 +65,7 @@ include:
>  .libcamera-ci.debian:12:
>    variables:
>      FDO_DISTRIBUTION_VERSION: 'bookworm'
> -    FDO_DISTRIBUTION_TAG: '2024-12-12.1'
> +    FDO_DISTRIBUTION_TAG: '2024-12-16.1'
> 
>  .libcamera-ci.debian:13:
>    variables:

Patch
diff mbox series

diff --git a/.gitlab-ci/setup-container.sh b/.gitlab-ci/setup-container.sh
index d2909c7..0658368 100755
--- a/.gitlab-ci/setup-container.sh
+++ b/.gitlab-ci/setup-container.sh
@@ -103,6 +103,9 @@  case $FDO_DISTRIBUTION_VERSION in
 'bookworm')
 	# libclang-rt-dev for the clang ASan runtime.
 	PKGS_LIBCAMERA_RUNTIME_MULTIARCH+=( libclang-rt-dev )
+	# For cam and lc-compliance
+	# libevent-dev cannot be used here, see build-libcamera-common.sh
+	PKGS_LIBCAMERA_RUNTIME_MULTIARCH+=( libevent-2.1-7 libevent-pthreads-2.1-7 )
 	;;
 'trixie')
 	# gcc 13 to expand compilation testing coverage.
diff --git a/gitlab-ci.yml b/gitlab-ci.yml
index 6d26680..f679d6e 100644
--- a/gitlab-ci.yml
+++ b/gitlab-ci.yml
@@ -65,7 +65,7 @@  include:
 .libcamera-ci.debian:12:
   variables:
     FDO_DISTRIBUTION_VERSION: 'bookworm'
-    FDO_DISTRIBUTION_TAG: '2024-12-12.1'
+    FDO_DISTRIBUTION_TAG: '2024-12-16.1'

 .libcamera-ci.debian:13:
   variables: