[dpdk-dev] [PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-29 Thread Wiles, Keith
On 4/29/15, 6:51 AM, "Thomas Monjalon" wrote: >2015-04-29 13:08 GMT+02:00 Gonzalez Monroy, Sergio >: >> On 29/04/2015 11:12, Thomas Monjalon wrote: >>> It seems this is the second version of your patch. >>> Please add v2 prefix and a changelog to ease review and >>> patch management. >>> As

[dpdk-dev] [PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-29 Thread Thomas Monjalon
2015-04-29 13:08 GMT+02:00 Gonzalez Monroy, Sergio : > On 29/04/2015 11:12, Thomas Monjalon wrote: >> It seems this is the second version of your patch. >> Please add v2 prefix and a changelog to ease review and >> patch management. >> As you probably know, it is explained here: >>

[dpdk-dev] [PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-29 Thread Thomas Monjalon
Hi Keith, Thanks for trying to improve maintainability. It seems this is the second version of your patch. Please add v2 prefix and a changelog to ease review and patch management. As you probably know, it is explained here: http://dpdk.org/dev#send 2015-04-28 19:03 GMT+02:00 Keith Wiles :

[dpdk-dev] [PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-29 Thread Gonzalez Monroy, Sergio
On 29/04/2015 11:12, Thomas Monjalon wrote: > Hi Keith, > > Thanks for trying to improve maintainability. > > It seems this is the second version of your patch. > Please add v2 prefix and a changelog to ease review and > patch management. > As you probably know, it is explained here: >

[dpdk-dev] [PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-28 Thread Keith Wiles
Trying to simplify the ifdefs in rte.app.mk to make the code more readable and maintainable by moving LDLIBS variable to use the same style as LDLIBS-y being used in the rest of the code. Added a new variable called EXTRA_LDLIBS to be used by example apps instead of using LDLIBS directly.