public inbox for gdb-testers@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Improve gdb_remote_download, remove gdb_download
@ 2016-04-05 18:35 sergiodj+buildbot
  2016-04-05 18:35 ` Failures on Fedora-i686, branch master sergiodj+buildbot
                   ` (18 more replies)
  0 siblings, 19 replies; 20+ messages in thread
From: sergiodj+buildbot @ 2016-04-05 18:35 UTC (permalink / raw)
  To: gdb-testers

*** TEST RESULTS FOR COMMIT 7817ea46148df2931cf00d17aeedb844b6e4e4db ***

Author: Simon Marchi <simon.marchi@ericsson.com>
Branch: master
Commit: 7817ea46148df2931cf00d17aeedb844b6e4e4db

Improve gdb_remote_download, remove gdb_download

This patch removes gdb_download in favor of gdb_remote_download, since
they are very close in functionality.  Also, in preparation for the
following patch about shared library handling during tests, it improves
gdb_remote_download so that it uses standard_output_file for any
destination board that is local, not only host.

If the destination board is remote, gdb_remote_download will use the
standard remote_download from DejaGnu, resulting in the file being
transferred on the remote system.

If the destination is local, gdb_remote_download will copy the file to
the standard test directory (found using standard_output_file).  Tcl's
file copy seems to handle gracefully cases where the source file is the
same as the destination, so I don't think it's necessary to check for
that case ourselves, as a previous version of the patch did.

I'd prefer to keep the name gdb_download instead of gdb_remote_download,
since I don't like the fact that gdb_remote_download implies that the
destination is remote, when it's not always the case.  However,
gdb_remote_download is used at many more places than gdb_download, so
it's easier to reuse that.  Also, since it's a wrapper around DejaGnu's
remote_download, it might be better to keep that name.  I don't know.

I ran the testsuite native, with native-gdbserver and with a
remote gdbserver, and didn't see any related failure.

gdb/testsuite/ChangeLog:

	* gdb.base/jit-so.exp: Use gdb_remote_download instead of
	gdb_download.  Use it even if the target is not remote.
	* gdb.base/jit.exp (compile_jit_test): Likewise.
	* lib/gdb.exp (gdb_remote_download): Copy files to the standard
	output directory if the destination board is local, otherwise use
	the standard remote_download from DejaGnu.
	(gdb_download): Remove.
	(gdb_load_shlibs): Use gdb_remote_download instead of
	gdb_download.
	* lib/gdbserver-support.exp (gdbserver_download_current_prog):
	Use gdb_remote_download instead of gdb_download.  Use it even if
	the target is not remote.
	* lib/mi-support.exp (mi_load_shlibs): Use gdb_remote_download
	instead of gdb_download.


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

end of thread, other threads:[~2016-04-06  5:44 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-05 18:35 [binutils-gdb] Improve gdb_remote_download, remove gdb_download sergiodj+buildbot
2016-04-05 18:35 ` Failures on Fedora-i686, branch master sergiodj+buildbot
2016-04-05 18:46 ` Failures on Fedora-x86_64-m32, " sergiodj+buildbot
2016-04-05 18:58 ` Failures on Fedora-x86_64-cc-with-index, " sergiodj+buildbot
2016-04-05 19:02 ` Failures on AIX-POWER7-plain, " sergiodj+buildbot
2016-04-05 19:03 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-04-05 19:16 ` Failures on Fedora-x86_64-m64, " sergiodj+buildbot
2016-04-05 19:39 ` Failures on Fedora-x86_64-native-extended-gdbserver-m32, " sergiodj+buildbot
2016-04-05 21:02 ` Failures on Debian-i686, " sergiodj+buildbot
2016-04-05 21:30 ` Failures on Debian-i686-native-extended-gdbserver, " sergiodj+buildbot
2016-04-05 21:49 ` Failures on Debian-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-04-05 22:23 ` Failures on Debian-x86_64-m64, " sergiodj+buildbot
2016-04-05 23:31 ` Failures on Debian-s390x-m64, " sergiodj+buildbot
2016-04-06  0:09 ` Failures on Debian-s390x-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-04-06  1:30 ` Failures on Fedora-ppc64be-cc-with-index, " sergiodj+buildbot
2016-04-06  1:51 ` Failures on Fedora-ppc64be-m64, " sergiodj+buildbot
2016-04-06  2:29 ` Failures on Fedora-ppc64be-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-04-06  4:21 ` Failures on Fedora-ppc64le-cc-with-index, " sergiodj+buildbot
2016-04-06  5:09 ` Failures on Fedora-ppc64le-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-04-06  5:44 ` Failures on Fedora-ppc64le-m64, " sergiodj+buildbot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).