27/11/2020 12:07, Nick Connolly: > Compiling with MinGW in --buildtype=debug produces a redefinition > error for strncasecmp. > > The root cause is that rte_os.h shouldn't be injecting POSIX definitions > into the environment. It is the applications responsibility to decide > how to handle missing functionality. > > Resolving this properly will require further work, but in the meantime > wrap all such definitions with #ifndef/#endif. This resolves the specific > issue with strncasecmp and handles similar issues that applications may > encounter. > > Fixes: e8428a9d89f1 ("eal/windows: add some basic functions and macros") > Cc: sta...@dpdk.org > > Signed-off-by: Nick Connolly <nick.conno...@mayadata.io>
Reported-by: David Marchand <david.march...@redhat.com> It looks as a good fix to me. I'm surprised there was no review. Applied, thanks