[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Thomas Monjalon
2016-07-18 15:54, Christian Ehrhardt: > Hi Sergio, > you might have seen that I had a similar version with awk in v2 IIRC. I > also had the secondary tmp file just like you now. > So, since it is so close to my old submission I wont object :-) > > Back then the discussion went for reduced build ti

[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Christian Ehrhardt
Hi Sergio, you might have seen that I had a similar version with awk in v2 IIRC. I also had the secondary tmp file just like you now. So, since it is so close to my old submission I wont object :-) Back then the discussion went for reduced build time dependencies and avoiding a second temp file, w

[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Sergio Gonzalez Monroy
On 18/07/2016 15:07, Thomas Monjalon wrote: > 2016-07-18 15:54, Christian Ehrhardt: >> Hi Sergio, >> you might have seen that I had a similar version with awk in v2 IIRC. I >> also had the secondary tmp file just like you now. >> So, since it is so close to my old submission I wont object :-) >> >>

[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Thomas Monjalon
2016-07-18 14:11, Sergio Gonzalez Monroy: > The sed syntax of '0,/regexp/' is GNU specific and fails with > non GNU sed in FreeBSD. > > To solve the issue we can use awk instead to remove duplicates. Christian, an opinion please? > Fixes: b2063f104db7 ("mk: filter duplicate configuration entries

[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Sergio Gonzalez Monroy
On 18/07/2016 14:54, Christian Ehrhardt wrote: > Hi Sergio, > you might have seen that I had a similar version with awk in v2 IIRC. > I also had the secondary tmp file just like you now. > So, since it is so close to my old submission I wont object :-) > > Back then the discussion went for reduced

[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Sergio Gonzalez Monroy
On 18/07/2016 14:25, Thomas Monjalon wrote: > 2016-07-18 14:11, Sergio Gonzalez Monroy: >> The sed syntax of '0,/regexp/' is GNU specific and fails with >> non GNU sed in FreeBSD. >> >> To solve the issue we can use awk instead to remove duplicates. > Christian, an opinion please? Sorry, forgot to

[dpdk-dev] [PATCH] mk: fix FreeBSD build

2016-07-18 Thread Sergio Gonzalez Monroy
The sed syntax of '0,/regexp/' is GNU specific and fails with non GNU sed in FreeBSD. To solve the issue we can use awk instead to remove duplicates. Fixes: b2063f104db7 ("mk: filter duplicate configuration entries") Signed-off-by: Sergio Gonzalez Monroy --- mk/rte.sdkconfig.mk | 8 +++- 1