Re: [PATCH v3 7/7] buildtools/chkincs: test headers for C++ compatibility

[Bruce Richardson]

On Thu, Feb 10, 2022 at 03:37:04PM +0100, David Marchand wrote:
> On Thu, Feb 10, 2022 at 3:05 PM Bruce Richardson
> <bruce.richard...@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.richard...@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
> 
Yes, will do.