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

2022-02-11 Thread Thomas Monjalon
11/02/2022 11:57, Bruce Richardson: > On Fri, Feb 11, 2022 at 10:31:38AM +, Bruce Richardson wrote: > > On Thu, Feb 10, 2022 at 10:58:37PM +0100, Thomas Monjalon wrote: > > > 10/02/2022 16:42, Bruce Richardson: > > > > +executable('chkincs-cpp', cpp_sources, +cpp_args: ['-include', > >

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

2022-02-11 Thread Bruce Richardson
On Fri, Feb 11, 2022 at 10:31:38AM +, Bruce Richardson wrote: > On Thu, Feb 10, 2022 at 10:58:37PM +0100, Thomas Monjalon wrote: > > 10/02/2022 16:42, Bruce Richardson: > > > +executable('chkincs-cpp', cpp_sources, +cpp_args: ['-include', > > > 'rte_config.h', cflags], +link_arg

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

2022-02-11 Thread Bruce Richardson
On Thu, Feb 10, 2022 at 10:58:37PM +0100, Thomas Monjalon wrote: > 10/02/2022 16:42, Bruce Richardson: > > +executable('chkincs-cpp', cpp_sources, +cpp_args: ['-include', > > 'rte_config.h', cflags], +link_args: dpdk_extra_ldflags, + > > include_directories: includes, +depen

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

2022-02-10 Thread Thomas Monjalon
10/02/2022 16:42, Bruce Richardson: > +executable('chkincs-cpp', cpp_sources, > +cpp_args: ['-include', 'rte_config.h', cflags], > +link_args: dpdk_extra_ldflags, > +include_directories: includes, > +dependencies: deps, > +link_whole: dpdk_static_libraries +

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

2022-02-10 Thread Bruce Richardson
Add support for checking each of our headers for issues when included in a C++ file. Signed-off-by: Bruce Richardson --- .ci/linux-build.sh | 1 + .github/workflows/build.yml| 2 +- buildtools/chkincs/main.cpp| 4 buildtools/chkincs/meson.build | 20 ++