Recheck-request: iol-intel-Functional Sorry about the spam!
On Tue, Apr 2, 2024 at 12:55 PM Tyler Retzlaff <roret...@linux.microsoft.com> wrote: > > Some libraries that could be built with MSVC were not being built. > > Switch from explicit include to exclude of libs to get immediate CI > coverage of libraries that already work with MSVC Windows builds. > > Signed-off-by: Tyler Retzlaff <roret...@linux.microsoft.com> > --- > lib/argparse/meson.build | 6 ++++++ > lib/dmadev/meson.build | 6 ++++++ > lib/mbuf/meson.build | 6 ++++++ > lib/mempool/meson.build | 6 ++++++ > lib/meson.build | 10 ---------- > lib/rcu/meson.build | 6 ++++++ > lib/stack/meson.build | 6 ++++++ > 7 files changed, 36 insertions(+), 10 deletions(-) > > diff --git a/lib/argparse/meson.build b/lib/argparse/meson.build > index b6a08ca..8ab4c40 100644 > --- a/lib/argparse/meson.build > +++ b/lib/argparse/meson.build > @@ -1,6 +1,12 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2024 HiSilicon Limited. > > +if is_ms_compiler > + build = false > + reason = 'not supported building with Visual Studio Toolset' > + subdir_done() > +endif > + > sources = files('rte_argparse.c') > headers = files('rte_argparse.h') > > diff --git a/lib/dmadev/meson.build b/lib/dmadev/meson.build > index 62b0650..e66dcb6 100644 > --- a/lib/dmadev/meson.build > +++ b/lib/dmadev/meson.build > @@ -1,6 +1,12 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2021 HiSilicon Limited. > > +if is_ms_compiler > + build = false > + reason = 'not supported building with Visual Studio Toolset' > + subdir_done() > +endif > + > sources = files('rte_dmadev.c', 'rte_dmadev_trace_points.c') > headers = files('rte_dmadev.h') > indirect_headers += files('rte_dmadev_core.h', 'rte_dmadev_trace_fp.h') > diff --git a/lib/mbuf/meson.build b/lib/mbuf/meson.build > index 0435c5e..2cee905 100644 > --- a/lib/mbuf/meson.build > +++ b/lib/mbuf/meson.build > @@ -1,6 +1,12 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2017 Intel Corporation > > +if is_ms_compiler > + build = false > + reason = 'not supported building with Visual Studio Toolset' > + subdir_done() > +endif > + > sources = files( > 'rte_mbuf.c', > 'rte_mbuf_ptype.c', > diff --git a/lib/mempool/meson.build b/lib/mempool/meson.build > index 8099a56..acce66c 100644 > --- a/lib/mempool/meson.build > +++ b/lib/mempool/meson.build > @@ -1,6 +1,12 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2017 Intel Corporation > > +if is_ms_compiler > + build = false > + reason = 'not supported building with Visual Studio Toolset' > + subdir_done() > +endif > + > extra_flags = [] > > foreach flag: extra_flags > diff --git a/lib/meson.build b/lib/meson.build > index 179a272..94d2b72 100644 > --- a/lib/meson.build > +++ b/lib/meson.build > @@ -68,16 +68,6 @@ libraries = [ > 'node', > ] > > -if is_ms_compiler > - libraries = [ > - 'log', > - 'kvargs', > - 'telemetry', > - 'eal', > - 'ring', > - ] > -endif > - > always_enable = [ > 'cmdline', > 'eal', > diff --git a/lib/rcu/meson.build b/lib/rcu/meson.build > index 09abc52..71143f5 100644 > --- a/lib/rcu/meson.build > +++ b/lib/rcu/meson.build > @@ -1,6 +1,12 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2018 Arm Limited > > +if is_ms_compiler > + build = false > + reason = 'not supported building with Visual Studio Toolset' > + subdir_done() > +endif > + > sources = files('rte_rcu_qsbr.c') > headers = files('rte_rcu_qsbr.h') > > diff --git a/lib/stack/meson.build b/lib/stack/meson.build > index 18177a7..7631a14 100644 > --- a/lib/stack/meson.build > +++ b/lib/stack/meson.build > @@ -1,6 +1,12 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2019 Intel Corporation > > +if is_ms_compiler > + build = false > + reason = 'not supported building with Visual Studio Toolset' > + subdir_done() > +endif > + > sources = files('rte_stack.c', 'rte_stack_std.c', 'rte_stack_lf.c') > headers = files('rte_stack.h') > # subheaders, not for direct inclusion by apps > -- > 1.8.3.1 >