Re: [PATCH] build/eal: add OS defines for C conditional checks

2021-12-14 Thread Bruce Richardson
On Tue, Dec 14, 2021 at 01:58:03AM +0300, Dmitry Kozlyuk wrote: > 2021-12-10 14:53 (UTC+), Bruce Richardson: > [...] > > Acked-by: Dmitry Kozlyuk > with one typo below and some considerations for the future in the bottom. > > > +Defines to Avoid Conditional Compilation > > +~

Re: [PATCH] build/eal: add OS defines for C conditional checks

2021-12-13 Thread Dmitry Kozlyuk
2021-12-10 14:53 (UTC+), Bruce Richardson: [...] Acked-by: Dmitry Kozlyuk with one typo below and some considerations for the future in the bottom. > +Defines to Avoid Conditional Compilation > + > + > +In many cases in DPDK, one wants to optionally co

Re: [PATCH] build/eal: add OS defines for C conditional checks

2021-12-12 Thread Jerin Jacob
On Fri, Dec 10, 2021 at 8:23 PM Bruce Richardson wrote: > > Define a set of macros in the build configuration to allow C runtime > code to check the current OS environment. This saves the user having to > use ifdefs for e.g. disabling particular tests on Windows. See included > documentation chang