* Re: gdb.mi/mi2-file.exp: ChangeLog, copyright, sync with mi-file.exp
@ 2004-06-30 1:52 Bob Rossi
2004-07-06 21:03 ` Bob Rossi
0 siblings, 1 reply; 4+ messages in thread
From: Bob Rossi @ 2004-06-30 1:52 UTC (permalink / raw)
To: gdb-patches
> >>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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: gdb.mi/mi2-file.exp: ChangeLog, copyright, sync with mi-file.exp
2004-06-30 1:52 gdb.mi/mi2-file.exp: ChangeLog, copyright, sync with mi-file.exp Bob Rossi
@ 2004-07-06 21:03 ` Bob Rossi
2004-07-07 15:30 ` Andrew Cagney
0 siblings, 1 reply; 4+ messages in thread
From: Bob Rossi @ 2004-07-06 21:03 UTC (permalink / raw)
To: gdb-patches; +Cc: cagney
Is this OK to commit? Andrew I think you approved it off of the list,
should I wait for public approval?
Thanks,
Bob Rossi
On Tue, Jun 29, 2004 at 09:51:37PM -0400, Bob Rossi wrote:
> > >>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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2004-07-08 1:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-06-30 1:52 gdb.mi/mi2-file.exp: ChangeLog, copyright, sync with mi-file.exp Bob Rossi
2004-07-06 21:03 ` Bob Rossi
2004-07-07 15:30 ` Andrew Cagney
2004-07-08 1:58 ` Bob Rossi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox