Re: [dpdk-dev] [PATCH v3 6/6] app/testpmd: enable testpmd on Windows

[Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>]

2021-04-14 10:16 (UTC-0700), Jie Zhou:
> On Tue, Apr 13, 2021 at 03:22:57PM -0700, Jie Zhou wrote:
> > On Tue, Apr 13, 2021 at 11:10:00PM +0300, Dmitry Kozlyuk wrote:  
> > > 2021-04-13 10:19 (UTC-0700), Jie Zhou:
> > > [...]  
> > > > diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
> > > > index fb7a3a8bd..9945adcf1 100644
> > > > --- a/app/test-pmd/cmdline_flow.c
> > > > +++ b/app/test-pmd/cmdline_flow.c
> > > > @@ -31,6 +31,12 @@
> > > >  
> > > >  #include "testpmd.h"
> > > >  
> > > > +#ifdef RTE_EXEC_ENV_WINDOWS
> > > > +#ifndef IPDEFTTL
> > > > +#define IPDEFTTL 64
> > > > +#endif
> > > > +#endif
> > > > +
> > > >  /** Parser token indices. */
> > > >  enum index {
> > > >  	/* Special tokens. */  
> > > 
> > > IPDEFTTL is used in some other apps and examples.
> > > If you follow Tal's advice and base your work on "Do not expose POSIX
> > > symbols" series, then please move this to <rte_os_shim.h>.  
> > 
> > I applied "eal/windows:do not expose POSIX symbols" per your and Tal's advices, but hit error LNK2019: unresolved external symbol strdup reference in function log_save_level". I am using clang version 10.0.0.  
> 
> [Update] Seems the problem is at applying the patch series, even though it showed all sucessfully applied, but somehow in eal_common_log.c the include of <rte_os_shim.h> was not added. After add include rte_os_shim.h, build completes.

Are you applying v9? It includes this exact fix.