Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate

Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
       [not found] <E1dq0Uz-00056v-Ti@kwanyin.sergiodj.net>
@ 2017-09-07 17:56 ` Palmer Dabbelt
  2017-09-07 18:00   ` H.J. Lu
  2017-09-07 18:52   ` Sergio Durigan Junior
  0 siblings, 2 replies; 13+ messages in thread
From: Palmer Dabbelt @ 2017-09-07 17:56 UTC (permalink / raw)
  To: gdb-buildbot; +Cc: gdb-patches

I think this wasn't my commits, as it seems to be broken as of 63c99141deff
("remove QStartupWithShell entry from NEWS::Changes in GDB 7.11").  Also, I
haven't touched anything in aarch64 GDB land

  https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2745

I can't find any other mention of this on "gdb-patches", though I'm not a GDB
maintainer (we don't have a RISC-V GDB port yet) so I had to poke around the
archives and therefor may be missing something.  According to here

  https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64?numbuilds=400

This has been broken since 34d16ea2a144 ("gdb.base/commands.exp: Test
loop_break and loop_continue in nested loops"), but that failure isn't relevant
to this one as it's a test failure not a build failure.

  https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2671

I poked around and there was at least one compilation failure in
between the one I just got emailed about and the test failure above, caused by
7022349d5c86 ("Stop assuming no-debug-info functions return int").

  https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2673/steps/compile%20gdb/logs/stdio

Given that there's two different failures in three commits and I'm pretty sure
I didn't cause this bug, I don't want to dig through the rest of the history to
figure out what's going on.  This has been broken for a while, is everyone else
just ignoring these messages?

On Thu, 07 Sep 2017 10:21:53 PDT (-0700), gdb-buildbot@sergiodj.net wrote:
> My lords, ladies, gentlemen, members of the public.
>
> It is a matter of great regret and sadness to inform you that commit:
>
> 	RISC-V: Support PCREL_* relocations agaist weak undefined symbols
> 	b1308d2c3749cc454f00b70768ee33724d919527
>
> might have made GDB unwell.  Since I am just your Butler BuildBot,
> I kindly ask that a human superior officer double-check this.
>
> Please note that if you are reading this message on gdb-patches, there might
> be other builders broken.
>
> You can find more details about the unfortunate breakage below.
>
> Cheers,
>
> Your GDB BuildBot.
>
> +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>
>
> Buildslave:
> 	ubuntu-aarch64-1
>
> Full Build URL:
> 	<http://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2748>
>
> Commit(s) tested:
> 	b1308d2c3749cc454f00b70768ee33724d919527
>
> Author(s) (in the same order as the commits):
> 	Palmer Dabbelt <palmer@dabbelt.com>
>
> Subject:
> 	RISC-V: Support PCREL_* relocations agaist weak undefined symbols
>
> Testsuite log (gdb.sum and gdb.log) URL(s):
> 	<http://gdb-build.sergiodj.net/results/Ubuntu-AArch64-native-gdbserver-m64/b1/b1308d2c3749cc454f00b70768ee33724d919527/>
>
> *** Failed to compiled GDB.  ***
> ============================
>
> +++ The full log is too big to be posted here.
> +++ These are the last 100 lines of it.
>
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE   ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xstrerror.c -o noasan/xstrerror.o; \
> 	else true; fi
> gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xstrerror.c -o xstrerror.o
> if [ x"" != x ]; then \
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE  ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xstrndup.c -o pic/xstrndup.o; \
> 	else true; fi
> if [ x"" != x ]; then \
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE  ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xvasprintf.c -o pic/xvasprintf.o; \
> 	else true; fi
> if [ x"" != x ]; then \
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE   ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xstrndup.c -o noasan/xstrndup.o; \
> 	else true; fi
> if [ x"" != x ]; then \
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE   ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xvasprintf.c -o noasan/xvasprintf.o; \
> 	else true; fi
> gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xstrndup.c -o xstrndup.o
> gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/xvasprintf.c -o xvasprintf.o
> if [ x"" != x ]; then \
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE  ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/setproctitle.c -o pic/setproctitle.o; \
> 	else true; fi
> if [ x"" != x ]; then \
> 	  gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE   ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/setproctitle.c -o noasan/setproctitle.o; \
> 	else true; fi
> gcc -c -DHAVE_CONFIG_H -g -O2  -I. -I../../../../binutils-gdb/gdb/gdbserver/../../libiberty/../include  -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic  -D_GNU_SOURCE ../../../../binutils-gdb/gdb/gdbserver/../../libiberty/setproctitle.c -o setproctitle.o
> rm -f ./libiberty.a pic/./libiberty.a noasan/./libiberty.a
> ar rc ./libiberty.a \
> 	  ./regex.o ./cplus-dem.o ./cp-demangle.o ./md5.o ./sha1.o ./alloca.o ./argv.o ./choose-temp.o ./concat.o ./cp-demint.o ./crc32.o ./d-demangle.o ./dwarfnames.o ./dyn-string.o ./fdmatch.o ./fibheap.o ./filename_cmp.o ./floatformat.o ./fnmatch.o ./fopen_unlocked.o ./getopt.o ./getopt1.o ./getpwd.o ./getruntime.o ./hashtab.o ./hex.o ./lbasename.o ./lrealpath.o ./make-relative-prefix.o ./make-temp-file.o ./objalloc.o ./obstack.o ./partition.o ./pexecute.o ./physmem.o ./pex-common.o ./pex-one.o ./pex-unix.o ./vprintf-support.o ./rust-demangle.o ./safe-ctype.o ./simple-object.o ./simple-object-coff.o ./simple-object-elf.o ./simple-object-mach-o.o ./simple-object-xcoff.o ./sort.o ./spaces.o ./splay-tree.o ./stack-limit.o ./strerror.o ./strsignal.o ./timeval-utils.o ./unlink-if-ordinary.o ./xasprintf.o ./xatexit.o ./xexit.o ./xmalloc.o ./xmemdup.o ./xstrdup.o ./xstrerror.o ./xstrndup.o ./xvasprintf.o  ./setproctitle.o
> ranlib ./libiberty.a
> if [ x"" != x ]; then \
> 	  cd pic; \
> 	  ar rc ./libiberty.a \
> 	    ./regex.o ./cplus-dem.o ./cp-demangle.o ./md5.o ./sha1.o ./alloca.o ./argv.o ./choose-temp.o ./concat.o ./cp-demint.o ./crc32.o ./d-demangle.o ./dwarfnames.o ./dyn-string.o ./fdmatch.o ./fibheap.o ./filename_cmp.o ./floatformat.o ./fnmatch.o ./fopen_unlocked.o ./getopt.o ./getopt1.o ./getpwd.o ./getruntime.o ./hashtab.o ./hex.o ./lbasename.o ./lrealpath.o ./make-relative-prefix.o ./make-temp-file.o ./objalloc.o ./obstack.o ./partition.o ./pexecute.o ./physmem.o ./pex-common.o ./pex-one.o ./pex-unix.o ./vprintf-support.o ./rust-demangle.o ./safe-ctype.o ./simple-object.o ./simple-object-coff.o ./simple-object-elf.o ./simple-object-mach-o.o ./simple-object-xcoff.o ./sort.o ./spaces.o ./splay-tree.o ./stack-limit.o ./strerror.o ./strsignal.o ./timeval-utils.o ./unlink-if-ordinary.o ./xasprintf.o ./xatexit.o ./xexit.o ./xmalloc.o ./xmemdup.o ./xstrdup.o ./xstrerror.o ./xstrndup.o ./xvasprintf.o  ./setproctitle.o; \
> 	  ranlib ./libiberty.a; \
> 	  cd ..; \
> 	else true; fi; \
> 	if [ x"" != x ]; then \
> 	  cd noasan; \
> 	  ar rc ./libiberty.a \
> 	    ./regex.o ./cplus-dem.o ./cp-demangle.o ./md5.o ./sha1.o ./alloca.o ./argv.o ./choose-temp.o ./concat.o ./cp-demint.o ./crc32.o ./d-demangle.o ./dwarfnames.o ./dyn-string.o ./fdmatch.o ./fibheap.o ./filename_cmp.o ./floatformat.o ./fnmatch.o ./fopen_unlocked.o ./getopt.o ./getopt1.o ./getpwd.o ./getruntime.o ./hashtab.o ./hex.o ./lbasename.o ./lrealpath.o ./make-relative-prefix.o ./make-temp-file.o ./objalloc.o ./obstack.o ./partition.o ./pexecute.o ./physmem.o ./pex-common.o ./pex-one.o ./pex-unix.o ./vprintf-support.o ./rust-demangle.o ./safe-ctype.o ./simple-object.o ./simple-object-coff.o ./simple-object-elf.o ./simple-object-mach-o.o ./simple-object-xcoff.o ./sort.o ./spaces.o ./splay-tree.o ./stack-limit.o ./strerror.o ./strsignal.o ./timeval-utils.o ./unlink-if-ordinary.o ./xasprintf.o ./xatexit.o ./xexit.o ./xmalloc.o ./xmemdup.o ./xstrdup.o ./xstrerror.o ./xstrndup.o ./xvasprintf.o  ./setproctitle.o; \
> 	  ranlib ./libiberty.a; \
> 	  cd ..; \
> 	else true; fi
> make[6]: Leaving directory `/home/qiyao/buildbot/ubuntu-aarch64-native-gdbserver-m64/build/gdb/gdbserver/build-libiberty-gdbserver'
> make[5]: Leaving directory `/home/qiyao/buildbot/ubuntu-aarch64-native-gdbserver-m64/build/gdb/gdbserver'
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o ax.o -MT ax.o -MMD -MP -MF .deps/ax.Tpo `echo " -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral " | sed "s/ -Wformat-nonliteral / -Wno-format-nonliteral /g"` ../../../binutils-gdb/gdb/gd
bs
>
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o btrace-common.o -MT btrace-common.o -MMD -MP -MF .deps/btrace-common.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/btrace-common.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o buffer.o -MT buffer.o -MMD -MP -MF .deps/buffer.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/buffer.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o cleanups.o -MT cleanups.o -MMD -MP -MF .deps/cleanups.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/cleanups.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o common-debug.o -MT common-debug.o -MMD -MP -MF .deps/common-debug.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/common-debug.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o common-exceptions.o -MT common-exceptions.o -MMD -MP -MF .deps/common-exceptions.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/common-exceptions.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o job-control.o -MT job-control.o -MMD -MP -MF .deps/job-control.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/job-control.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o common-regcache.o -MT common-regcache.o -MMD -MP -MF .deps/common-regcache.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/common-regcache.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o common-utils.o -MT common-utils.o -MMD -MP -MF .deps/common-utils.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/common-utils.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o debug.o -MT debug.o -MMD -MP -MF .deps/debug.Tpo ../../../binutils-gdb/gdb/gdbserver/debug.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o dll.o -MT dll.o -MMD -MP -MF .deps/dll.Tpo ../../../binutils-gdb/gdb/gdbserver/dll.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o errors.o -MT errors.o -MMD -MP -MF .deps/errors.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/errors.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o environ.o -MT environ.o -MMD -MP -MF .deps/environ.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/environ.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o event-loop.o -MT event-loop.o -MMD -MP -MF .deps/event-loop.Tpo ../../../binutils-gdb/gdb/gdbserver/event-loop.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o fileio.o -MT fileio.o -MMD -MP -MF .deps/fileio.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/fileio.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o filestuff.o -MT filestuff.o -MMD -MP -MF .deps/filestuff.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/filestuff.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o format.o -MT format.o -MMD -MP -MF .deps/format.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/format.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o gdb_vecs.o -MT gdb_vecs.o -MMD -MP -MF .deps/gdb_vecs.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/gdb_vecs.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o hostio.o -MT hostio.o -MMD -MP -MF .deps/hostio.Tpo ../../../binutils-gdb/gdb/gdbserver/hostio.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o inferiors.o -MT inferiors.o -MMD -MP -MF .deps/inferiors.Tpo ../../../binutils-gdb/gdb/gdbserver/inferiors.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o mem-break.o -MT mem-break.o -MMD -MP -MF .deps/mem-break.Tpo ../../../binutils-gdb/gdb/gdbserver/mem-break.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o new-op.o -MT new-op.o -MMD -MP -MF .deps/new-op.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/new-op.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o notif.o -MT notif.o -MMD -MP -MF .deps/notif.Tpo ../../../binutils-gdb/gdb/gdbserver/notif.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o print-utils.o -MT print-utils.o -MMD -MP -MF .deps/print-utils.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/print-utils.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o ptid.o -MT ptid.o -MMD -MP -MF .deps/ptid.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/ptid.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o regcache.o -MT regcache.o -MMD -MP -MF .deps/regcache.Tpo ../../../binutils-gdb/gdb/gdbserver/regcache.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o remote-utils.o -MT remote-utils.o -MMD -MP -MF .deps/remote-utils.Tpo ../../../binutils-gdb/gdb/gdbserver/remote-utils.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o rsp-low.o -MT rsp-low.o -MMD -MP -MF .deps/rsp-low.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/rsp-low.c
> g++ -x c++ -std=gnu++11 -g -O2    -I. -I../../../binutils-gdb/gdb/gdbserver -I../../../binutils-gdb/gdb/gdbserver/../common -I../../../binutils-gdb/gdb/gdbserver/../regformats -I../../../binutils-gdb/gdb/gdbserver/.. -I../../../binutils-gdb/gdb/gdbserver/../../include -I../../../binutils-gdb/gdb/gdbserver/../gnulib/import -Ibuild-gnulib-gdbserver/import  -Wall -Wpointer-arith -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable -Wno-sign-compare -Wno-narrowing -Wno-error=maybe-uninitialized -Wformat-nonliteral -Werror -DGDBSERVER -c -o selftest.o -MT selftest.o -MMD -MP -MF .deps/selftest.Tpo ../../../binutils-gdb/gdb/gdbserver/../common/selftest.c
> In file included from /usr/include/string.h:640:0,
>                  from build-gnulib-gdbserver/import/string.h:41,
>                  from ../../../binutils-gdb/gdb/gdbserver/../common/common-defs.h:56,
>                  from ../../../binutils-gdb/gdb/gdbserver/server.h:22,
>                  from ../../../binutils-gdb/gdb/gdbserver/regcache.c:19:
> In function void* memset(void*, int, size_t),
>     inlined from regcache* init_register_cache(regcache*, const target_desc*, unsigned char*) at ../../../binutils-gdb/gdb/gdbserver/regcache.c:150:50:
> /usr/include/aarch64-linux-gnu/bits/string3.h:81:32: error: call to __warn_memset_zero_len declared with attribute warning: memset used with constant zero length parameter; this could be due to transposed parameters [-Werror]
>        __warn_memset_zero_len ();
>                                 ^
> In function void* memset(void*, int, size_t),
>     inlined from regcache* get_thread_regcache(thread_info*, int) at ../../../binutils-gdb/gdb/gdbserver/regcache.c:57:60:
> /usr/include/aarch64-linux-gnu/bits/string3.h:81:32: error: call to __warn_memset_zero_len declared with attribute warning: memset used with constant zero length parameter; this could be due to transposed parameters [-Werror]
>        __warn_memset_zero_len ();
>                                 ^
> cc1plus: all warnings being treated as errors
> make[4]: *** [regcache.o] Error 1
> make[4]: *** Waiting for unfinished jobs....
> make[4]: Leaving directory `/home/qiyao/buildbot/ubuntu-aarch64-native-gdbserver-m64/build/gdb/gdbserver'
> make[3]: *** [subdir_do] Error 1
> make[2]: *** [all] Error 2
> make[3]: Leaving directory `/home/qiyao/buildbot/ubuntu-aarch64-native-gdbserver-m64/build/gdb'
> make[2]: Leaving directory `/home/qiyao/buildbot/ubuntu-aarch64-native-gdbserver-m64/build/gdb'
> make[1]: *** [all-gdb] Error 2
> make: *** [all] Error 2
> make[1]: Leaving directory `/home/qiyao/buildbot/ubuntu-aarch64-native-gdbserver-m64/build'
> ============================


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 17:56 ` Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate Palmer Dabbelt
@ 2017-09-07 18:00   ` H.J. Lu
  2017-09-07 18:57     ` Sergio Durigan Junior
  2017-09-07 23:35     ` Joel Brobecker
  2017-09-07 18:52   ` Sergio Durigan Junior
  1 sibling, 2 replies; 13+ messages in thread
From: H.J. Lu @ 2017-09-07 18:00 UTC (permalink / raw)
  To: Palmer Dabbelt; +Cc: gdb-buildbot, GDB

On Thu, Sep 7, 2017 at 10:56 AM, Palmer Dabbelt <palmer@dabbelt.com> wrote:
> I think this wasn't my commits, as it seems to be broken as of 63c99141deff
> ("remove QStartupWithShell entry from NEWS::Changes in GDB 7.11").  Also, I
> haven't touched anything in aarch64 GDB land
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2745
>
> I can't find any other mention of this on "gdb-patches", though I'm not a GDB
> maintainer (we don't have a RISC-V GDB port yet) so I had to poke around the
> archives and therefor may be missing something.  According to here
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64?numbuilds=400
>
> This has been broken since 34d16ea2a144 ("gdb.base/commands.exp: Test
> loop_break and loop_continue in nested loops"), but that failure isn't relevant
> to this one as it's a test failure not a build failure.
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2671
>
> I poked around and there was at least one compilation failure in
> between the one I just got emailed about and the test failure above, caused by
> 7022349d5c86 ("Stop assuming no-debug-info functions return int").
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2673/steps/compile%20gdb/logs/stdio
>
> Given that there's two different failures in three commits and I'm pretty sure
> I didn't cause this bug, I don't want to dig through the rest of the history to
> figure out what's going on.  This has been broken for a while, is everyone else
> just ignoring these messages?
>

This bot is next to useless.  I just ignore it.

This bot shouldn't spam people when the new commit doesn't change
the old error.


-- 
H.J.


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 18:00   ` H.J. Lu
@ 2017-09-07 18:57     ` Sergio Durigan Junior
  2017-09-07 19:32       ` H.J. Lu
  2017-09-07 23:35     ` Joel Brobecker
  1 sibling, 1 reply; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-07 18:57 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Palmer Dabbelt, GDB

On Thursday, September 07 2017, H. J. Lu wrote:

> On Thu, Sep 7, 2017 at 10:56 AM, Palmer Dabbelt <palmer@dabbelt.com> wrote:
>> I think this wasn't my commits, as it seems to be broken as of 63c99141deff
>> ("remove QStartupWithShell entry from NEWS::Changes in GDB 7.11").  Also, I
>> haven't touched anything in aarch64 GDB land
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2745
>>
>> I can't find any other mention of this on "gdb-patches", though I'm not a GDB
>> maintainer (we don't have a RISC-V GDB port yet) so I had to poke around the
>> archives and therefor may be missing something.  According to here
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64?numbuilds=400
>>
>> This has been broken since 34d16ea2a144 ("gdb.base/commands.exp: Test
>> loop_break and loop_continue in nested loops"), but that failure isn't relevant
>> to this one as it's a test failure not a build failure.
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2671
>>
>> I poked around and there was at least one compilation failure in
>> between the one I just got emailed about and the test failure above, caused by
>> 7022349d5c86 ("Stop assuming no-debug-info functions return int").
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2673/steps/compile%20gdb/logs/stdio
>>
>> Given that there's two different failures in three commits and I'm pretty sure
>> I didn't cause this bug, I don't want to dig through the rest of the history to
>> figure out what's going on.  This has been broken for a while, is everyone else
>> just ignoring these messages?
>>
>
> This bot is next to useless.  I just ignore it.

Funny, I have another opinion of what is really useless.

> This bot shouldn't spam people when the new commit doesn't change
> the old error.

The bot is not spamming anyone, it had a bug in the way it used to
determine whether the breakage email should be sent.  The bug has been
fixed now.

These breakage e-mails have caught many issues over time, so I still
think they're very useful.  Of course, if the GDB community decides, I
can take the bot down at any time.  If this happens, I suposse you must
have a really better solution for CI.

Anyway, it's a work-in-progress and has a few drawbacks and limitations,
but overall I think people are reasonably satisfied with it.

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 18:57     ` Sergio Durigan Junior
@ 2017-09-07 19:32       ` H.J. Lu
  2017-09-07 19:41         ` Sergio Durigan Junior
  0 siblings, 1 reply; 13+ messages in thread
From: H.J. Lu @ 2017-09-07 19:32 UTC (permalink / raw)
  To: Sergio Durigan Junior; +Cc: Palmer Dabbelt, GDB

On Thu, Sep 7, 2017 at 11:57 AM, Sergio Durigan Junior
<sergiodj@redhat.com> wrote:
> On Thursday, September 07 2017, H. J. Lu wrote:
>
>> On Thu, Sep 7, 2017 at 10:56 AM, Palmer Dabbelt <palmer@dabbelt.com> wrote:
>>> I think this wasn't my commits, as it seems to be broken as of 63c99141deff
>>> ("remove QStartupWithShell entry from NEWS::Changes in GDB 7.11").  Also, I
>>> haven't touched anything in aarch64 GDB land
>>>
>>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2745
>>>
>>> I can't find any other mention of this on "gdb-patches", though I'm not a GDB
>>> maintainer (we don't have a RISC-V GDB port yet) so I had to poke around the
>>> archives and therefor may be missing something.  According to here
>>>
>>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64?numbuilds=400
>>>
>>> This has been broken since 34d16ea2a144 ("gdb.base/commands.exp: Test
>>> loop_break and loop_continue in nested loops"), but that failure isn't relevant
>>> to this one as it's a test failure not a build failure.
>>>
>>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2671
>>>
>>> I poked around and there was at least one compilation failure in
>>> between the one I just got emailed about and the test failure above, caused by
>>> 7022349d5c86 ("Stop assuming no-debug-info functions return int").
>>>
>>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2673/steps/compile%20gdb/logs/stdio
>>>
>>> Given that there's two different failures in three commits and I'm pretty sure
>>> I didn't cause this bug, I don't want to dig through the rest of the history to
>>> figure out what's going on.  This has been broken for a while, is everyone else
>>> just ignoring these messages?
>>>
>>
>> This bot is next to useless.  I just ignore it.
>
> Funny, I have another opinion of what is really useless.
>
>> This bot shouldn't spam people when the new commit doesn't change
>> the old error.
>
> The bot is not spamming anyone, it had a bug in the way it used to
> determine whether the breakage email should be sent.  The bug has been
> fixed now.
>
> These breakage e-mails have caught many issues over time, so I still
> think they're very useful.  Of course, if the GDB community decides, I
> can take the bot down at any time.  If this happens, I suposse you must
> have a really better solution for CI.
>
> Anyway, it's a work-in-progress and has a few drawbacks and limitations,
> but overall I think people are reasonably satisfied with it.
>

Does it build for a target for any new commits which are unrelated to the
target?  For example,  when I make some changes to bfd/elfxx-x86.h,
will it triggers a gdb build for Sparc?

-- 
H.J.


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 19:32       ` H.J. Lu
@ 2017-09-07 19:41         ` Sergio Durigan Junior
  2017-09-07 19:52           ` H.J. Lu
  0 siblings, 1 reply; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-07 19:41 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Palmer Dabbelt, GDB

On Thursday, September 07 2017, H. J. Lu wrote:

> Does it build for a target for any new commits which are unrelated to the
> target?  For example,  when I make some changes to bfd/elfxx-x86.h,
> will it triggers a gdb build for Sparc?

We don't have a Sparc buildslave, but every commit related to GDB that
touches a specific set of files:

  # Don't build commits touching only these directories
  files_ignored_re = re.compile ("(binutils/|cpu/|elfcpp/|gas/|gold/|gprof/|ld/|texinfo/|gdb/doc/).*")

will be built on *all* builders we have.

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 19:41         ` Sergio Durigan Junior
@ 2017-09-07 19:52           ` H.J. Lu
  2017-09-07 19:54             ` Sergio Durigan Junior
  2017-09-07 19:57             ` Sergio Durigan Junior
  0 siblings, 2 replies; 13+ messages in thread
From: H.J. Lu @ 2017-09-07 19:52 UTC (permalink / raw)
  To: Sergio Durigan Junior; +Cc: Palmer Dabbelt, GDB

On Thu, Sep 7, 2017 at 12:41 PM, Sergio Durigan Junior
<sergiodj@redhat.com> wrote:
> On Thursday, September 07 2017, H. J. Lu wrote:
>
>> Does it build for a target for any new commits which are unrelated to the
>> target?  For example,  when I make some changes to bfd/elfxx-x86.h,
>> will it triggers a gdb build for Sparc?
>
> We don't have a Sparc buildslave, but every commit related to GDB that
> touches a specific set of files:
>
>   # Don't build commits touching only these directories
>   files_ignored_re = re.compile ("(binutils/|cpu/|elfcpp/|gas/|gold/|gprof/|ld/|texinfo/|gdb/doc/).*")
>
> will be built on *all* builders we have.

When there is a build failure in sparc-nat.c, does a bfd/elfxx-x86.h change
trigger a new build?

-- 
H.J.


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 19:52           ` H.J. Lu
@ 2017-09-07 19:54             ` Sergio Durigan Junior
  2017-09-07 19:57             ` Sergio Durigan Junior
  1 sibling, 0 replies; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-07 19:54 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Palmer Dabbelt, GDB

On Thursday, September 07 2017, H. J. Lu wrote:

> On Thu, Sep 7, 2017 at 12:41 PM, Sergio Durigan Junior
> <sergiodj@redhat.com> wrote:
>> On Thursday, September 07 2017, H. J. Lu wrote:
>>
>>> Does it build for a target for any new commits which are unrelated to the
>>> target?  For example,  when I make some changes to bfd/elfxx-x86.h,
>>> will it triggers a gdb build for Sparc?
>>
>> We don't have a Sparc buildslave, but every commit related to GDB that
>> touches a specific set of files:
>>
>>   # Don't build commits touching only these directories
>>   files_ignored_re = re.compile ("(binutils/|cpu/|elfcpp/|gas/|gold/|gprof/|ld/|texinfo/|gdb/doc/).*")
>>
>> will be built on *all* builders we have.
>
> When there is a build failure in sparc-nat.c, does a bfd/elfxx-x86.h change
> trigger a new build?

No.

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 19:52           ` H.J. Lu
  2017-09-07 19:54             ` Sergio Durigan Junior
@ 2017-09-07 19:57             ` Sergio Durigan Junior
  1 sibling, 0 replies; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-07 19:57 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Palmer Dabbelt, GDB

On Thursday, September 07 2017, H. J. Lu wrote:

> On Thu, Sep 7, 2017 at 12:41 PM, Sergio Durigan Junior
> <sergiodj@redhat.com> wrote:
>> On Thursday, September 07 2017, H. J. Lu wrote:
>>
>>> Does it build for a target for any new commits which are unrelated to the
>>> target?  For example,  when I make some changes to bfd/elfxx-x86.h,
>>> will it triggers a gdb build for Sparc?
>>
>> We don't have a Sparc buildslave, but every commit related to GDB that
>> touches a specific set of files:
>>
>>   # Don't build commits touching only these directories
>>   files_ignored_re = re.compile ("(binutils/|cpu/|elfcpp/|gas/|gold/|gprof/|ld/|texinfo/|gdb/doc/).*")
>>
>> will be built on *all* builders we have.
>
> When there is a build failure in sparc-nat.c, does a bfd/elfxx-x86.h change
> trigger a new build?

Actually, yes.  Every new commit will trigger a new build, but if the
breakage is still present, no new breakage e-mails will be sent.

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 18:00   ` H.J. Lu
  2017-09-07 18:57     ` Sergio Durigan Junior
@ 2017-09-07 23:35     ` Joel Brobecker
  2017-09-08  2:10       ` H.J. Lu
  1 sibling, 1 reply; 13+ messages in thread
From: Joel Brobecker @ 2017-09-07 23:35 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Palmer Dabbelt, gdb-buildbot, GDB

> This bot is next to useless.  I just ignore it.
> 
> This bot shouldn't spam people when the new commit doesn't change
> the old error.

Personally, I think it is great that we have a buildbot that tests
so many configurations and platforms. This is a great tool and
while there are issues today, I really don't think we should be
ignoring them. The right thing to do was to report/ask here, which
led to Sergio immediately jumping on it.  Thank you Sergio!

This buildbot is really there to help us out discover regressions
soon after we commit, when it's easier to investigate and adjust,
or revert if necessary. While we get annoyed sometime, it's still
going to benefit us all. Let's encourage its improvement, and offer
our help to its development team, rather than denigrate it.

-- 
Joel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 23:35     ` Joel Brobecker
@ 2017-09-08  2:10       ` H.J. Lu
  2017-09-08  5:41         ` Sergio Durigan Junior
  0 siblings, 1 reply; 13+ messages in thread
From: H.J. Lu @ 2017-09-08  2:10 UTC (permalink / raw)
  To: Joel Brobecker; +Cc: Palmer Dabbelt, gdb-buildbot, GDB

On Thu, Sep 7, 2017 at 4:34 PM, Joel Brobecker <brobecker@adacore.com> wrote:
>> This bot is next to useless.  I just ignore it.
>>
>> This bot shouldn't spam people when the new commit doesn't change
>> the old error.
>
> Personally, I think it is great that we have a buildbot that tests
> so many configurations and platforms. This is a great tool and
> while there are issues today, I really don't think we should be
> ignoring them. The right thing to do was to report/ask here, which
> led to Sergio immediately jumping on it.  Thank you Sergio!
>
> This buildbot is really there to help us out discover regressions
> soon after we commit, when it's easier to investigate and adjust,
> or revert if necessary. While we get annoyed sometime, it's still
> going to benefit us all. Let's encourage its improvement, and offer
> our help to its development team, rather than denigrate it.

Sure.

FWIW, I got another 5 bogus messsages from:

https://sourceware.org/ml/binutils/2017-09/msg00070.html

which has nothing to do with GDB build failure.  I will put it in
my spam filter if it doesn't stop.


-- 
H.J.


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-08  2:10       ` H.J. Lu
@ 2017-09-08  5:41         ` Sergio Durigan Junior
  0 siblings, 0 replies; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-08  5:41 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Joel Brobecker, Palmer Dabbelt, GDB

On Thursday, September 07 2017, H. J. Lu wrote:

> On Thu, Sep 7, 2017 at 4:34 PM, Joel Brobecker <brobecker@adacore.com> wrote:
>>> This bot is next to useless.  I just ignore it.
>>>
>>> This bot shouldn't spam people when the new commit doesn't change
>>> the old error.
>>
>> Personally, I think it is great that we have a buildbot that tests
>> so many configurations and platforms. This is a great tool and
>> while there are issues today, I really don't think we should be
>> ignoring them. The right thing to do was to report/ask here, which
>> led to Sergio immediately jumping on it.  Thank you Sergio!
>>
>> This buildbot is really there to help us out discover regressions
>> soon after we commit, when it's easier to investigate and adjust,
>> or revert if necessary. While we get annoyed sometime, it's still
>> going to benefit us all. Let's encourage its improvement, and offer
>> our help to its development team, rather than denigrate it.
>
> Sure.
>
> FWIW, I got another 5 bogus messsages from:
>
> https://sourceware.org/ml/binutils/2017-09/msg00070.html
>
> which has nothing to do with GDB build failure.  I will put it in
> my spam filter if it doesn't stop.

I'm sorry to hear that.  Things should be more stable now.  Please let
me know if you still receive wrong e-mails.

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 17:56 ` Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate Palmer Dabbelt
  2017-09-07 18:00   ` H.J. Lu
@ 2017-09-07 18:52   ` Sergio Durigan Junior
  2017-09-10  5:54     ` Sergio Durigan Junior
  1 sibling, 1 reply; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-07 18:52 UTC (permalink / raw)
  To: Palmer Dabbelt; +Cc: gdb-patches, Yao Qi

On Thursday, September 07 2017, Palmer Dabbelt wrote:

> I think this wasn't my commits, as it seems to be broken as of 63c99141deff
> ("remove QStartupWithShell entry from NEWS::Changes in GDB 7.11").  Also, I
> haven't touched anything in aarch64 GDB land
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2745
>
> I can't find any other mention of this on "gdb-patches", though I'm not a GDB
> maintainer (we don't have a RISC-V GDB port yet) so I had to poke around the
> archives and therefor may be missing something.  According to here
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64?numbuilds=400
>
> This has been broken since 34d16ea2a144 ("gdb.base/commands.exp: Test
> loop_break and loop_continue in nested loops"), but that failure isn't relevant
> to this one as it's a test failure not a build failure.
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2671
>
> I poked around and there was at least one compilation failure in
> between the one I just got emailed about and the test failure above, caused by
> 7022349d5c86 ("Stop assuming no-debug-info functions return int").
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2673/steps/compile%20gdb/logs/stdio
>
> Given that there's two different failures in three commits and I'm pretty sure
> I didn't cause this bug, I don't want to dig through the rest of the history to
> figure out what's going on.  This has been broken for a while, is everyone else
> just ignoring these messages?

Hi Palmer,

Thanks for investigating these issues.

There's been a bunch of compilation errors on this builder,
unfortunately.  The last one is caused by commit
f7000548a2b79d7e5cb924468117ca4245e6b820, from Yao.

Yao, could you please take a look at the error?  You can see the build
logs here:

  https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2696/steps/compile%20gdb/logs/stdio

I apologize for the wrong emails, I have found a bug in the code that
determines whether the breakage email should be sent, and fixed it.
Things should be more stable now.

Thanks,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate
  2017-09-07 18:52   ` Sergio Durigan Junior
@ 2017-09-10  5:54     ` Sergio Durigan Junior
  0 siblings, 0 replies; 13+ messages in thread
From: Sergio Durigan Junior @ 2017-09-10  5:54 UTC (permalink / raw)
  To: Palmer Dabbelt; +Cc: gdb-patches, Yao Qi

On Thursday, September 07 2017, I wrote:

> On Thursday, September 07 2017, Palmer Dabbelt wrote:
>
>> I think this wasn't my commits, as it seems to be broken as of 63c99141deff
>> ("remove QStartupWithShell entry from NEWS::Changes in GDB 7.11").  Also, I
>> haven't touched anything in aarch64 GDB land
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2745
>>
>> I can't find any other mention of this on "gdb-patches", though I'm not a GDB
>> maintainer (we don't have a RISC-V GDB port yet) so I had to poke around the
>> archives and therefor may be missing something.  According to here
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64?numbuilds=400
>>
>> This has been broken since 34d16ea2a144 ("gdb.base/commands.exp: Test
>> loop_break and loop_continue in nested loops"), but that failure isn't relevant
>> to this one as it's a test failure not a build failure.
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2671
>>
>> I poked around and there was at least one compilation failure in
>> between the one I just got emailed about and the test failure above, caused by
>> 7022349d5c86 ("Stop assuming no-debug-info functions return int").
>>
>>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2673/steps/compile%20gdb/logs/stdio
>>
>> Given that there's two different failures in three commits and I'm pretty sure
>> I didn't cause this bug, I don't want to dig through the rest of the history to
>> figure out what's going on.  This has been broken for a while, is everyone else
>> just ignoring these messages?
>
> Hi Palmer,
>
> Thanks for investigating these issues.
>
> There's been a bunch of compilation errors on this builder,
> unfortunately.  The last one is caused by commit
> f7000548a2b79d7e5cb924468117ca4245e6b820, from Yao.
>
> Yao, could you please take a look at the error?  You can see the build
> logs here:
>
>   https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2696/steps/compile%20gdb/logs/stdio

FWIW, I sent a patch that fixes this:

  <https://sourceware.org/ml/gdb-patches/2017-09/msg00264.html>

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2017-09-10  5:54 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <E1dq0Uz-00056v-Ti@kwanyin.sergiodj.net>
2017-09-07 17:56 ` Oh dear. I regret to inform you that commit b1308d2c3749cc454f00b70768ee33724d919527 might be unfortunate Palmer Dabbelt
2017-09-07 18:00   ` H.J. Lu
2017-09-07 18:57     ` Sergio Durigan Junior
2017-09-07 19:32       ` H.J. Lu
2017-09-07 19:41         ` Sergio Durigan Junior
2017-09-07 19:52           ` H.J. Lu
2017-09-07 19:54             ` Sergio Durigan Junior
2017-09-07 19:57             ` Sergio Durigan Junior
2017-09-07 23:35     ` Joel Brobecker
2017-09-08  2:10       ` H.J. Lu
2017-09-08  5:41         ` Sergio Durigan Junior
2017-09-07 18:52   ` Sergio Durigan Junior
2017-09-10  5:54     ` Sergio Durigan Junior

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox