Re: [ovs-dev] [PATCH] travis: fix DPDK build

2014-09-10 Thread Thomas Graf
On 09/10/14 at 04:22pm, Daniele Di Proietto wrote: > That’s great, > > I see that -Wl,--whole-archive -Wl,-ldl -Wl,--no-whole-archive is used > when building ovs_client. That should not happen on master > (-Wl,--whole-archive is used only when we need to link all the pmd > drivers, i.e. when linki

Re: [ovs-dev] [PATCH] travis: fix DPDK build

2014-09-10 Thread Daniele Di Proietto
On 9/10/14, 7:54 AM, "Thomas Graf" wrote: >On 09/09/14 at 10:45pm, Daniele Di Proietto wrote: >> I¹ll try to figure out what¹s wrong an post another patch. > >Daniele, > >I have a patchset to properly enable the DPDK build, fix CFLAGS >and disable some errors to make -Werror work. The only thing

Re: [ovs-dev] [PATCH] travis: fix DPDK build

2014-09-10 Thread Thomas Graf
On 09/09/14 at 10:45pm, Daniele Di Proietto wrote: > I¹ll try to figure out what¹s wrong an post another patch. Daniele, I have a patchset to properly enable the DPDK build, fix CFLAGS and disable some errors to make -Werror work. The only thing left is resolving this linker error: /bin/bash ./l

Re: [ovs-dev] [PATCH] travis: fix DPDK build

2014-09-10 Thread Thomas Graf
On 09/09/14 at 03:17pm, Daniele Di Proietto wrote: > Signed-off-by: Daniele Di Proietto > --- > I didn't test the fix. If it works "netdev-dpdk.o" should appear in the travis > output (on master it doesn't) > > Thanks, > > Daniele > --- > .travis.yml | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [ovs-dev] [PATCH] travis: fix DPDK build

2014-09-09 Thread Daniele Di Proietto
On 9/9/14, 3:40 PM, "Ben Pfaff" wrote: >On Tue, Sep 09, 2014 at 03:17:00PM -0700, Daniele Di Proietto wrote: >> Signed-off-by: Daniele Di Proietto >> --- >> I didn't test the fix. If it works "netdev-dpdk.o" should appear in the >>travis >> output (on master it doesn't) > >I think that it corr

Re: [ovs-dev] [PATCH] travis: fix DPDK build

2014-09-09 Thread Ben Pfaff
On Tue, Sep 09, 2014 at 03:17:00PM -0700, Daniele Di Proietto wrote: > Signed-off-by: Daniele Di Proietto > --- > I didn't test the fix. If it works "netdev-dpdk.o" should appear in the travis > output (on master it doesn't) I think that it corrects the build, but the build fails because DPDK isn

[ovs-dev] [PATCH] travis: fix DPDK build

2014-09-09 Thread Daniele Di Proietto
Signed-off-by: Daniele Di Proietto --- I didn't test the fix. If it works "netdev-dpdk.o" should appear in the travis output (on master it doesn't) Thanks, Daniele --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index fd21e30..b66f82