[PATCH v3 7/7] buildtools/chkincs: test headers for C++ compatibility
David Marchand
david.marchand at redhat.com
Thu Feb 10 15:37:04 CET 2022
On Thu, Feb 10, 2022 at 3:05 PM Bruce Richardson
<bruce.richardson at intel.com> wrote:
>
> Add support for checking each of our headers for issues when included in
> a C++ file.
>
> Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
> ---
> .ci/linux-build.sh | 1 +
> buildtools/chkincs/main.cpp | 4 ++++
> buildtools/chkincs/meson.build | 20 ++++++++++++++++++++
> 3 files changed, 25 insertions(+)
> create mode 100644 buildtools/chkincs/main.cpp
>
> diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh
> index c10c1a8ab5..67d68535e0 100755
> --- a/.ci/linux-build.sh
> +++ b/.ci/linux-build.sh
> @@ -74,6 +74,7 @@ fi
>
> if [ "$BUILD_32BIT" = "true" ]; then
> OPTS="$OPTS -Dc_args=-m32 -Dc_link_args=-m32"
> + OPTS="$OPTS -Dcpp_args=-m32 -Dcpp_link_args=-m32"
> export PKG_CONFIG_LIBDIR="/usr/lib32/pkgconfig"
> fi
>
Even though check_includes=true is not forced for 32bits in
test-meson-builds.sh, I sometimes enable more checks for some targets.
Can this change be applied to test-meson-builds.sh too?
https://git.dpdk.org/dpdk/tree/devtools/test-meson-builds.sh#n249
--
David Marchand
More information about the dev
mailing list