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

2015-05-01 Thread Bruce Richardson
On Thu, Apr 30, 2015 at 05:33:36PM +0100, Wiles, Keith wrote: > > > On 4/30/15, 11:22 AM, "Richardson, Bruce" > wrote: > > >On Thu, Apr 30, 2015 at 02:31:13PM +, Wiles, Keith wrote: > >> > >> > >> On 4/30/15, 8:38 AM, "Olivier MATZ" wrote: > >> > >> >Hi Keith, > >> > > >> >On 04/30/2015

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

2015-04-30 Thread Olivier MATZ
Hi, On 04/30/2015 04:31 PM, Wiles, Keith wrote: > > > On 4/30/15, 8:38 AM, "Olivier MATZ" wrote: > >> Hi Keith, >> >> On 04/30/2015 03:24 PM, Wiles, Keith wrote: >>> >>> >>> On 4/30/15, 4:45 AM, "Olivier MATZ" wrote: >>> Hi Keith, Thank you for submitting a clean-up. Please see so

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

2015-04-30 Thread Bruce Richardson
On Thu, Apr 30, 2015 at 02:31:13PM +, Wiles, Keith wrote: > > > On 4/30/15, 8:38 AM, "Olivier MATZ" wrote: > > >Hi Keith, > > > >On 04/30/2015 03:24 PM, Wiles, Keith wrote: > >> > >> > >> On 4/30/15, 4:45 AM, "Olivier MATZ" wrote: > >> > >>> Hi Keith, > >>> > >>> Thank you for submitting a

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

2015-04-30 Thread Wiles, Keith
On 4/30/15, 11:22 AM, "Richardson, Bruce" wrote: >On Thu, Apr 30, 2015 at 02:31:13PM +, Wiles, Keith wrote: >> >> >> On 4/30/15, 8:38 AM, "Olivier MATZ" wrote: >> >> >Hi Keith, >> > >> >On 04/30/2015 03:24 PM, Wiles, Keith wrote: >> >> >> >> >> >> On 4/30/15, 4:45 AM, "Olivier MATZ" wr

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

2015-04-30 Thread Olivier MATZ
Hi Keith, On 04/30/2015 03:24 PM, Wiles, Keith wrote: > > > On 4/30/15, 4:45 AM, "Olivier MATZ" wrote: > >> Hi Keith, >> >> Thank you for submitting a clean-up. Please see some comments below. >> >> On 04/29/2015 05:25 PM, Keith Wiles wrote: >>> Trying to simplify the ifdefs in rte.app.mk to make

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

2015-04-30 Thread Wiles, Keith
On 4/30/15, 8:38 AM, "Olivier MATZ" wrote: >Hi Keith, > >On 04/30/2015 03:24 PM, Wiles, Keith wrote: >> >> >> On 4/30/15, 4:45 AM, "Olivier MATZ" wrote: >> >>> Hi Keith, >>> >>> Thank you for submitting a clean-up. Please see some comments below. >>> >>> On 04/29/2015 05:25 PM, Keith Wiles wro

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

2015-04-30 Thread Wiles, Keith
On 4/30/15, 4:45 AM, "Olivier MATZ" wrote: >Hi Keith, > >Thank you for submitting a clean-up. Please see some comments below. > >On 04/29/2015 05:25 PM, Keith Wiles wrote: >> Trying to simplify the ifdefs in rte.app.mk to make the code >> more readable and maintainable by moving LDLIBS variable

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

2015-04-30 Thread Olivier MATZ
Hi Keith, Thank you for submitting a clean-up. Please see some comments below. On 04/29/2015 05:25 PM, Keith Wiles wrote: > 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

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

2015-04-29 Thread Thomas Monjalon
2015-04-29 10:25, Keith Wiles: > -# Copyright(c) 2010-2014 Intel Corporation. All rights reserved. > -# Copyright(c) 2014 6WIND S.A. > +# Copyright(c) 2010-2015 Intel Corporation. All rights reserved. > +# Copyright(c) 2015 6WIND S.A. A correct update of copyright dates should be - 2014 +

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

2015-04-29 Thread Wiles, Keith
On 4/29/15, 12:17 PM, "Thomas Monjalon" wrote: >2015-04-29 10:25, Keith Wiles: >> -# Copyright(c) 2010-2014 Intel Corporation. All rights reserved. >> -# Copyright(c) 2014 6WIND S.A. >> +# Copyright(c) 2010-2015 Intel Corporation. All rights reserved. >> +# Copyright(c) 2015 6WIND S.A.

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

2015-04-29 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. Signed-