Re: [patch 2/2] Fix linking on non-x86* after libgdb.a removal

[Pedro Alves <palves@redhat.com>]

On 01/09/2012 09:01 PM, Jan Kratochvil wrote:
> @@ -72,8 +72,7 @@ core_file_command (char *filename, int from_tty)
>  {
>    dont_repeat ();		/* Either way, seems bogus.  */
>  
> -  if (core_target == NULL)
> -    error (_("GDB can't read core files on this machine."));
> +  gdb_assert (core_target != NULL);

I think we should stop expecting it in the testsuite too.  Any objections to this?

gdb/testsuite/
2012-01-10  Pedro Alves  <palves@redhat.com>

	* gdb.base/default.exp (core-file): Don't expect "GDB can't read
	core files on this machine" anymore.

---

 gdb/testsuite/gdb.base/default.exp |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gdb/testsuite/gdb.base/default.exp b/gdb/testsuite/gdb.base/default.exp
index 74d2d2d..9677396 100644
--- a/gdb/testsuite/gdb.base/default.exp
+++ b/gdb/testsuite/gdb.base/default.exp
@@ -102,7 +102,7 @@ gdb_test "commands" "Argument required .one or more breakpoint numbers...*" "com
 gdb_test "condition" "Argument required .breakpoint number.*" "condition"

 #test core-file
-gdb_test "core-file" "No core file now.|GDB can't read core files on this machine." "core-file"
+gdb_test "core-file" "No core file now." "core-file"
 #test delete "d" abbreviation
 gdb_test_no_output "d" "delete \"d\" abbreviation"
 #test delete