[v9,01/10] lib: build libraries that testpmd depends on

Message ID 1619805162-10684-2-git-send-email-jizh@linux.microsoft.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers
Series app/testpmd: enable testpmd on Windows |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Jie Zhou April 30, 2021, 5:52 p.m. UTC
  Enable building libraries that testpmd depends on

Signed-off-by: Jie Zhou <jizh@microsoft.com>
Signed-off-by: Jie Zhou <jizh@linux.microsoft.com>
---
 lib/meson.build | 7 +++++++
 1 file changed, 7 insertions(+)
  

Comments

Tal Shnaiderman May 3, 2021, 5:36 p.m. UTC | #1
> Subject: [PATCH v9 01/10] lib: build libraries that testpmd depends on
> 
> External email: Use caution opening links or attachments
> 
> 
> Enable building libraries that testpmd depends on
> 

You should mention the change is for Windows only.

> Signed-off-by: Jie Zhou <jizh@microsoft.com>
> Signed-off-by: Jie Zhou <jizh@linux.microsoft.com>
> ---
>  lib/meson.build | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/lib/meson.build b/lib/meson.build index c9a20f65b..2d499b238
> 100644
> --- a/lib/meson.build
> +++ b/lib/meson.build
> @@ -76,8 +76,15 @@ if is_windows
>              'ethdev',
>              'pci',
>              'cmdline',
> +            'metrics',
>              'hash',
> +            'timer',
> +            'bitratestats',
>              'cfgfile',
> +            'gro',
> +            'gso',
> +            'latencystats',
> +            'pdump',
>      ] # only supported libraries for windows  endif
> 
> --
> 2.30.0.vfs.0.2
  
Jie Zhou May 3, 2021, 9:38 p.m. UTC | #2
On Mon, May 03, 2021 at 05:36:30PM +0000, Tal Shnaiderman wrote:
> > Subject: [PATCH v9 01/10] lib: build libraries that testpmd depends on
> > 
> > External email: Use caution opening links or attachments
> > 
> > 
> > Enable building libraries that testpmd depends on
> > 
> 
> You should mention the change is for Windows only.

Will explicitly mention the change is for Windows only in V10.

> 
> > Signed-off-by: Jie Zhou <jizh@microsoft.com>
> > Signed-off-by: Jie Zhou <jizh@linux.microsoft.com>
> > ---
> >  lib/meson.build | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/lib/meson.build b/lib/meson.build index c9a20f65b..2d499b238
> > 100644
> > --- a/lib/meson.build
> > +++ b/lib/meson.build
> > @@ -76,8 +76,15 @@ if is_windows
> >              'ethdev',
> >              'pci',
> >              'cmdline',
> > +            'metrics',
> >              'hash',
> > +            'timer',
> > +            'bitratestats',
> >              'cfgfile',
> > +            'gro',
> > +            'gso',
> > +            'latencystats',
> > +            'pdump',
> >      ] # only supported libraries for windows  endif
> > 
> > --
> > 2.30.0.vfs.0.2
  

Patch

diff --git a/lib/meson.build b/lib/meson.build
index c9a20f65b..2d499b238 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -76,8 +76,15 @@  if is_windows
             'ethdev',
             'pci',
             'cmdline',
+            'metrics',
             'hash',
+            'timer',
+            'bitratestats',
             'cfgfile',
+            'gro',
+            'gso',
+            'latencystats',
+            'pdump',
     ] # only supported libraries for windows
 endif