01/03/2019 15:17, Bruce Richardson: > On Fri, Mar 01, 2019 at 03:03:02PM +0100, Thomas Monjalon wrote: > > 01/03/2019 08:18, Anand Rawat: > > > +elif host_machine.system() == 'windows' + > > > dpdk_conf.set('RTE_EXEC_ENV_WINDOWS', 1) > > > > For consistency, it should RTE_EXEC_ENV_WINAPP. > > > For this one you can partially blame me - early internal review suggested > changing to winapp, but I suggested holding off on the change for community > consensus. Personally, I would prefer a little inconsistency and keep it > as windows for clarity. In future, I also think we should just rename > linxuapp to linux, and bsdapp to freebsd!
OK I agree to remove APP suffixes. > However, for now, if the majority prefer winapp, I'm ok with it. [Though I > can't help reading it as winamp half the time! No llamas involved here > though!]