[ovs-dev] [PATCH] travis: Fix clang build for DPDK-2.0.

Daniele Di Proietto diproiettod at vmware.com
Fri May 8 16:08:51 UTC 2015


I've tested it on a branch and it works now.
Thanks for the fix!

Acked-by: Daniele Di Proietto <diproiettod at vmware.com>

> On 8 May 2015, at 09:28, Kavanagh, Mark B <mark.b.kavanagh at intel.com> wrote:
> 
>> -Wno-cast-align is a CFLAG, not a configure option.
>> 
>> Signed-off-by: Joe Stringer <joestringer at nicira.com>
>> ---
>> .travis/build.sh | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/.travis/build.sh b/.travis/build.sh
>> index c7796e1..6dfc9fe 100755
>> --- a/.travis/build.sh
>> +++ b/.travis/build.sh
>> @@ -74,7 +74,7 @@ if [ "$DPDK" ]; then
>>    install_dpdk $DPDK_VER
>>    if [ "$CC" = "clang" ]; then
>>        # Disregard cast alignment errors until DPDK is fixed
>> -        EXTRA_OPTS="$EXTRA_OPTS -Wno-cast-align"
>> +        CFLAGS="$CFLAGS -Wno-cast-align"
> 
> Hi Joe,
> 
> I was on the fence about where to put this flag - ultimately, I decided that if it were passed at the configure stage, then any subsequent 'make's wouldn't need the additional flag.
> However, I fully agree with this change.
> 
> Thanks,
> Mark
> 
>>    fi
>>    EXTRA_OPTS="$EXTRA_OPTS --with-dpdk=./dpdk-$DPDK_VER/build"
>> elif [ "$CC" != "clang" ]; then
>> --
>> 2.1.4




More information about the dev mailing list