Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Michael Elizabeth Chastain <chastain@cygnus.com>
To: gdb-patches@sources.redhat.com, jmoore@cygnus.com
Subject: Re: [RFA] Fix for gdb-gnats bug #14
Date: Fri, 16 Feb 2001 13:44:00 -0000	[thread overview]
Message-ID: <200102162144.NAA01284@bosch.cygnus.com> (raw)

Hi John,

I'm having trouble applying this patch.  The original file has a lot
of TAB characters in it, but in your patch, they are expanded to spaces.
This makes the patch not apply, and it also makes the new code incorrect
(these TABs are literal characters in regular expressions).

Could you please check your source carefully to make sure that you have
the right characters in the new source lines 213-214:

  \ [ SPACE TAB \ ] *

Once the source file is correct, be careful with cut-and-paste, and
check your mailer.

Also the first line of the ChangeLog starts with eight spaces; please make
that a tab.  I wouldn't mention that except I am asking you to re-make
the patch anyways.

Michael

===

The fix was tested on Red Hat Linux 7.0 with 2.4 kernel.

2001-02-16  John Moore  <jmoore@redhat.com>

        * gdb.base/commands.exp (infrun_breakpoint_command_test):
	Removed non-expected items following a "step" command
	(see gnats bug report for gdb #14).

Index: gdb.base/commands.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/commands.exp,v
retrieving revision 1.6
diff -p -u -w -r1.6 commands.exp
--- commands.exp        2000/10/24 18:36:32     1.6
+++ commands.exp        2001/02/16 17:48:18
@@ -210,17 +210,8 @@ proc infrun_breakpoint_command_test {} {
        gdb_test "continue" \
                "Breakpoint \[0-9\]*, factorial \\(value=5\\).*at.*
 \[0-9\]*\[     \]*if \\(value > 1\\) \{.*
-\[0-9\]*\[     \]*value \\*= factorial \\(value - 1\\);.*
-factorial \\(value=4\\) at.*\[0-9\]*\[         \]*if \\(value > 1\\) \{.*
-\[0-9\]*\[     \]*value \\*= factorial \\(value - 1\\);.*
-factorial \\(value=3\\) at .*
-\[0-9\]*\[     \]*if \\(value > 1\\) \{.*
-#0  factorial \\(value=3\\).*
-#1  \[0-9a-fx\]* in factorial \\(value=4\\).*
-#2  \[0-9a-fx\]* in factorial \\(value=5\\).*
-#3  \[0-9a-fx\]* in factorial \\(value=6\\).*
-#4  \[0-9a-fx\]* in main \\(.*\\).*" \
-               "continue in infrun_breakpoint_command_test";
+\[0-9\]*\[     \]*value \\*= factorial \\(value - 1\\);.*" \
+               "continue in infrun_breakpoint_command_test"
     }

     gdb_stop_suppressing_tests;




             reply	other threads:[~2001-02-16 13:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-02-16 13:44 Michael Elizabeth Chastain [this message]
2001-02-16 13:56 ` John R. Moore
  -- strict thread matches above, loose matches on Subject: below --
2001-02-18 15:57 Michael Elizabeth Chastain
2001-02-16 17:54 Michael Elizabeth Chastain
2001-02-16 17:19 Michael Elizabeth Chastain
2001-02-16 17:36 ` Michael Snyder
     [not found] <200102170027.QAA01551@bosch.cygnus.com>
2001-02-16 16:44 ` John R. Moore
2001-02-16 10:04 John R. Moore
2001-02-16 14:40 ` John R. Moore

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200102162144.NAA01284@bosch.cygnus.com \
    --to=chastain@cygnus.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=jmoore@cygnus.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox