Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Bob Rossi <bob@brasko.net>
To: gdb-patches@sources.redhat.com
Subject: Re: gdb.mi/mi2-file.exp: ChangeLog, copyright, sync with mi-file.exp
Date: Wed, 30 Jun 2004 01:52:00 -0000	[thread overview]
Message-ID: <20040630015136.GD4791@white> (raw)

> >>I am going to update the mi-file.exp to test my new functionality. I
> >>>am going to make mi2-file.exp the way it was before I changed it.
> >>>So, I should have a changelog only for mi-file.exp.
> >>>   mi2-file.exp will be reverted to as it was before my change.
> >>>   mi-file.exp will test the new command -file-list-exec-source-files.
> >
> >
> >Generally we don't retroactively edit ChangeLog's, we only add
> >new entries.
> >
> >I would do:
> >
> >(1) Revert the change to mi2-file.exp.  Use a normal ChangeLog
> >    entry for this, which says: "mi2-file.exp: Reverted undocumented
> >    change from 2004-NN-NN".  Send it to gdb-patches and ask
> >    Andrew or Elena to approve it.
> 
> Yes, that's approved.
> 
> >(2) Write a new patch for mi-file.exp, with a normal ChangeLog
> >    entry, and remember to hit the copyright years.
> 
> Again, that's approved.

Here is the patch, ready to approve? It basically reverts mi2-file.exp
to version 1.1, except changing the name of the test which was
originally wrong ( thanks to cut/paste ).

Also, update mi-file.exp to have the test for
-file-list-exec-source-files.

2004-06-29  Bob Rossi  <bob_rossi@cox.net>

	* gdb.mi/mi2-file.exp: Revert to version 1.1, except changed name of
	test from test_tbreak_creation_and_listing to
	test_file_list_exec_source_file
	* gdb.mi/mi-file.exp: Add test for -file-list-exec-source-files
	Changed name of test from test_tbreak_creation_and_listing to
	test_file_list_exec_source_file

Index: gdb.mi/mi-file.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.mi/mi-file.exp,v
retrieving revision 1.1
diff -w -u -r1.1 mi-file.exp
--- gdb.mi/mi-file.exp	2 Apr 2003 22:10:35 -0000	1.1
+++ gdb.mi/mi-file.exp	30 Jun 2004 01:42:45 -0000
@@ -1,4 +1,4 @@
-#   Copyright 1999 Free Software Foundation, Inc.
+#   Copyright 1999, 2004 Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -47,7 +47,7 @@
 mi_gdb_reinitialize_dir $srcdir/$subdir
 mi_gdb_load ${binfile}
 
-proc test_tbreak_creation_and_listing {} {
+proc test_file_list_exec_source_file {} {
     global srcfile
     global srcdir
     global subdir
@@ -59,7 +59,17 @@
                "request path info of current source file (${srcfile})"
 }
 
-test_tbreak_creation_and_listing
+proc test_file_list_exec_source_files {} {
+    global srcfile
+
+    # get the path and absolute path to the current executable
+    mi_gdb_test "222-file-list-exec-source-files" \
+	    "222\\\^done,files=\\\[\{file=\".*/${srcfile}\",fullname=\"/.*/${srcfile}\"\},\{file=\".*\"\},\{file=\".*\"\},\{file=\".*\"\},\{file=\".*\"\}\\\]" \
+              "Getting a list of source files."
+}
+
+test_file_list_exec_source_file
+test_file_list_exec_source_files
 
 mi_gdb_exit
 return 0
Index: gdb.mi/mi2-file.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.mi/mi2-file.exp,v
retrieving revision 1.2
diff -w -u -r1.2 mi2-file.exp
--- gdb.mi/mi2-file.exp	10 Jun 2004 20:05:45 -0000	1.2
+++ gdb.mi/mi2-file.exp	30 Jun 2004 01:42:45 -0000
@@ -1,4 +1,4 @@
-#   Copyright 1999 Free Software Foundation, Inc.
+#   Copyright 1999, 2004 Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -59,17 +59,7 @@
                "request path info of current source file (${srcfile})"
 }
 
-proc test_file_list_exec_source_files {} {
-    global srcfile
-
-    # get the path and absolute path to the current executable
-    mi_gdb_test "222-file-list-exec-source-files" \
-	    "222\\\^done,files=\\\[\{file=\".*/${srcfile}\",fullname=\"/.*/${srcfile}\"\},\{file=\".*\"\},\{file=\".*\"\},\{file=\".*\"\},\{file=\".*\"\}\\\]" \
-              "Getting a list of source files."
-}
-
 test_file_list_exec_source_file
-test_file_list_exec_source_files
 
 mi_gdb_exit
 return 0


             reply	other threads:[~2004-06-30  1:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-30  1:52 Bob Rossi [this message]
2004-07-06 21:03 ` Bob Rossi
2004-07-07 15:30   ` Andrew Cagney
2004-07-08  1:58     ` Bob Rossi

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=20040630015136.GD4791@white \
    --to=bob@brasko.net \
    --cc=gdb-patches@sources.redhat.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