[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