[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-14 Thread Olivier MATZ
Hi Keith, On 05/13/2015 04:43 PM, Wiles, Keith wrote: > > > On 5/13/15, 9:28 AM, "Olivier MATZ" wrote: > >> >> On 05/13/2015 04:04 PM, Wiles, Keith wrote: >>> >>> >>> On 5/13/15, 8:56 AM, "Olivier MATZ" wrote: >>> Hi Keith, On 05/13/2015 03:17 PM, Wiles, Keith wrote: >>> >>

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Olivier MATZ
On 05/13/2015 04:04 PM, Wiles, Keith wrote: > > > On 5/13/15, 8:56 AM, "Olivier MATZ" wrote: > >> Hi Keith, >> >> On 05/13/2015 03:17 PM, Wiles, Keith wrote: > > endif # ifeq ($(NO_AUTOLIBS),) > > -LDLIBS += $(CPU_LDLIBS) > +LDLIBS += $(_LDLIBS-y) $(EXTRA_LDLIBS) > >>>

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Olivier MATZ
Hi Keith, On 05/13/2015 03:17 PM, Wiles, Keith wrote: >>> >>>endif # ifeq ($(NO_AUTOLIBS),) >>> >>> -LDLIBS += $(CPU_LDLIBS) >>> +LDLIBS += $(_LDLIBS-y) $(EXTRA_LDLIBS) >>> >> >> As discussed in the previous mail, all things that are about >> EXTRA_LDLIBS should be moved in the second patch. T

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Wiles, Keith
On 5/13/15, 9:28 AM, "Olivier MATZ" wrote: > >On 05/13/2015 04:04 PM, Wiles, Keith wrote: >> >> >> On 5/13/15, 8:56 AM, "Olivier MATZ" wrote: >> >>> Hi Keith, >>> >>> On 05/13/2015 03:17 PM, Wiles, Keith wrote: >> >> endif # ifeq ($(NO_AUTOLIBS),) >> >> -LDLIBS += $(CPU_LDL

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Wiles, Keith
On 5/13/15, 9:28 AM, "Olivier MATZ" wrote: > >On 05/13/2015 04:04 PM, Wiles, Keith wrote: >> >> >> On 5/13/15, 8:56 AM, "Olivier MATZ" wrote: >> >>> Hi Keith, >>> >>> On 05/13/2015 03:17 PM, Wiles, Keith wrote: >> >> endif # ifeq ($(NO_AUTOLIBS),) >> >> -LDLIBS += $(CPU_LDL

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Wiles, Keith
On 5/13/15, 8:56 AM, "Olivier MATZ" wrote: >Hi Keith, > >On 05/13/2015 03:17 PM, Wiles, Keith wrote: endif # ifeq ($(NO_AUTOLIBS),) -LDLIBS += $(CPU_LDLIBS) +LDLIBS += $(_LDLIBS-y) $(EXTRA_LDLIBS) >>> >>> As discussed in the previous mail, all things that are ab

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Wiles, Keith
On 5/13/15, 2:41 AM, "Olivier MATZ" wrote: >Hi, > >On 05/12/2015 09:11 PM, Keith Wiles wrote: >> 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. The new int

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-13 Thread Olivier MATZ
Hi, On 05/12/2015 09:11 PM, Keith Wiles wrote: > 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. The new internal variable _LDLIBS should > not be used outside of

[dpdk-dev] [PATCH v8 1/2] mk:Simplify the ifdefs in rte.app.mk

2015-05-12 Thread Keith Wiles
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. The new internal variable _LDLIBS should not be used outside of the rte.app.mk file. Signed-off-by: Keith Wiles --- mk