Jakub Sitnicki <j...@redhat.com> writes:
> Hey Aaron,
>
> On Wed, 11 Jul 2018 15:55:34 -0400
> 0-day Robot <ro...@bytheb.org> wrote:
>
>> Bleep bloop. Greetings Jakub Sitnicki, I am a robot and I have tried out
>> your patch.
>> Thanks for your contribution.
>>
>> I encountered some error that I wasn't expecting. See the details below.
>>
>>
>> build:
>> depbase=`echo utilities/ovs-dpctl.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\
>> gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I ./include -I ./lib -I
>> ./lib -Wstrict-prototypes -Wall -Wextra -Wno-sign-compare -Wpointer-arith
>> -Wformat -Wformat-security -Wswitch-enum -Wunused-parameter
>> -Wbad-function-cast -Wcast-align -Wstrict-prototypes -Wold-style-definition
>> -Wmissing-prototypes -Wmissing-field-initializers -fno-strict-aliasing
>> -Wshadow -Werror -MT utilities/ovs-dpctl.o -MD -MP -MF $depbase.Tpo -c -o
>> utilities/ovs-dpctl.o utilities/ovs-dpctl.c &&\
>> mv -f $depbase.Tpo $depbase.Po
>> /bin/sh ./libtool --tag=CC --mode=link gcc -std=gnu99 -Wstrict-prototypes
>> -Wall -Wextra -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security
>> -Wswitch-enum -Wunused-parameter -Wbad-function-cast -Wcast-align
>> -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes
>> -Wmissing-field-initializers -fno-strict-aliasing -Wshadow -Werror -o
>> utilities/ovs-dpctl utilities/ovs-dpctl.o lib/libopenvswitch.la -lpthread
>> -lrt -lm -lunbound
>> libtool: link: gcc -std=gnu99 -Wstrict-prototypes -Wall -Wextra
>> -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security -Wswitch-enum
>> -Wunused-parameter -Wbad-function-cast -Wcast-align -Wstrict-prototypes
>> -Wold-style-definition -Wmissing-prototypes -Wmissing-field-initializers
>> -fno-strict-aliasing -Wshadow -Werror -o utilities/ovs-dpctl
>> utilities/ovs-dpctl.o lib/.libs/libopenvswitch.a -lssl -lcrypto -lcap-ng
>> -lpthread -lrt -lm -lunbound
>> depbase=`echo utilities/ovs-ofctl.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\
>> gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I ./include -I ./lib -I
>> ./lib -Wstrict-prototypes -Wall -Wextra -Wno-sign-compare -Wpointer-arith
>> -Wformat -Wformat-security -Wswitch-enum -Wunused-parameter
>> -Wbad-function-cast -Wcast-align -Wstrict-prototypes -Wold-style-definition
>> -Wmissing-prototypes -Wmissing-field-initializers -fno-strict-aliasing
>> -Wshadow -Werror -MT utilities/ovs-ofctl.o -MD -MP -MF $depbase.Tpo -c -o
>> utilities/ovs-ofctl.o utilities/ovs-ofctl.c &&\
>> mv -f $depbase.Tpo $depbase.Po
>> /bin/sh ./libtool --tag=CC --mode=link gcc -std=gnu99 -Wstrict-prototypes
>> -Wall -Wextra -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security
>> -Wswitch-enum -Wunused-parameter -Wbad-function-cast -Wcast-align
>> -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes
>> -Wmissing-field-initializers -fno-strict-aliasing -Wshadow -Werror -o
>> utilities/ovs-ofctl utilities/ovs-ofctl.o ofproto/libofproto.la
>> lib/libopenvswitch.la -lpthread -lrt -lm -lunbound
>> libtool: link: gcc -std=gnu99 -Wstrict-prototypes -Wall -Wextra
>> -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security -Wswitch-enum
>> -Wunused-parameter -Wbad-function-cast -Wcast-align -Wstrict-prototypes
>> -Wold-style-definition -Wmissing-prototypes -Wmissing-field-initializers
>> -fno-strict-aliasing -Wshadow -Werror -o utilities/ovs-ofctl
>> utilities/ovs-ofctl.o ofproto/.libs/libofproto.a
>> /var/lib/jenkins/jobs/upstream_build_from_pw/workspace/lib/.libs/libsflow.a
>> lib/.libs/libopenvswitch.a -lssl -lcrypto -lcap-ng -lpthread -lrt -lm
>> -lunbound
>> depbase=`echo utilities/ovs-vsctl.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\
>> gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I ./include -I ./lib -I
>> ./lib -Wstrict-prototypes -Wall -Wextra -Wno-sign-compare -Wpointer-arith
>> -Wformat -Wformat-security -Wswitch-enum -Wunused-parameter
>> -Wbad-function-cast -Wcast-align -Wstrict-prototypes -Wold-style-definition
>> -Wmissing-prototypes -Wmissing-field-initializers -fno-strict-aliasing
>> -Wshadow -Werror -MT utilities/ovs-vsctl.o -MD -MP -MF $depbase.Tpo -c -o
>> utilities/ovs-vsctl.o utilities/ovs-vsctl.c &&\
>> mv -f $depbase.Tpo $depbase.Po
>> In file included from utilities/ovs-vsctl.c:49:0:
>> ./lib/table.h:86:13: error: initializer element is not constant
>> (struct table_style) { TF_LIST, CF_STRING, true, JSSF_SORT, 0 }
>> ^
>> utilities/ovs-vsctl.c:85:41: note: in expansion of macro
>> ‘TABLE_STYLE_DEFAULT’
>> static struct table_style table_style = TABLE_STYLE_DEFAULT;
>> ^
>> make[2]: *** [utilities/ovs-vsctl.o] Error 1
>> make[2]: Leaving directory
>> `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace'
>> make[1]: *** [all-recursive] Error 1
>> make[1]: Leaving directory
>> `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace'
>> make: *** [all] Error 2
>
> Wich GCC version is it? I'm having trouble reproducing the failed
> build. Event if I set CFLAGS=-std=gnu99 (which I assume is something
> specific to the robot builds) I'm not getting this error with:
>
> gcc version 7.3.1 20180303 (Red Hat 7.3.1-5) (GCC), or
> clang version 5.0.2 (tags/RELEASE_502/final)
The build server is stock RHEL 7.5, so
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
I'll ask if we can possibly upgrade it (or look into using a docker
container for the build)
> BTW. The robot has quite spammed me with the same error report. Is there
> any way to disable duplicate notifications?
An excellent suggestion. I'll work on it.
> Thanks,
> Jakub
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
