[libcamera-ci] [PATCH v2] Enable `cpp_debugstl`

Kieran Bingham kieran.bingham at ideasonboard.com
Tue Dec 10 11:10:50 CET 2024


Quoting Barnabás Pőcze (2024-12-09 17:11:28)
> Meson's `cpp_debugstl` built-in option enables extra checks
> in libstdc++ and libc++, such as iterator invalidation tests,
> bounds checking in the `operator[]` of multiple types, etc.
> by setting `GLIBCXX_DEBUG` and `_LIBCPP_HARDENING_MODE`.
> 
> Enable it in the same cases ASan is enabled as they are
> somewhat similar debugging tools.
> 
> Signed-off-by: Barnabás Pőcze <barnabas.pocze at ideasonboard.com>
> ---
> https://gitlab.freedesktop.org/pobrn/libcamera/-/pipelines/1326230
> 
> Changes in v2:
>   * do not rebuild containers
>   * note that it is enabled when ASan is enabled

Sounds good to me.


Acked-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> ---
>  gitlab-ci.yml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/gitlab-ci.yml b/gitlab-ci.yml
> index ea038ec..c78235e 100644
> --- a/gitlab-ci.yml
> +++ b/gitlab-ci.yml
> @@ -15,6 +15,7 @@ variables:
>      -D android=enabled
>      -D b_sanitize=address
>      -D cam=enabled
> +    -D cpp_debugstl=true
>      -D documentation=enabled
>      -D gstreamer=enabled
>      -D lc-compliance=enabled
> @@ -389,6 +390,7 @@ test-unit:
>      MESON_OPTIONS: >-
>        -D b_sanitize=address
>        -D cam=disabled
> +      -D cpp_debugstl=true
>        -D documentation=disabled
>        -D gstreamer=enabled
>        -D lc-compliance=disabled
> --
> 2.47.1


More information about the libcamera-devel mailing list