* [pushed/master/gdb-7.9-branch] gdb/testsuite: Make clean mostlyclean should not delete *.py.
@ 2015-01-13 15:33 Joel Brobecker
0 siblings, 0 replies; only message in thread
From: Joel Brobecker @ 2015-01-13 15:33 UTC (permalink / raw)
To: gdb-patches
Hello,
A sanity-check in my release scripts caught something: After having
created the tarballs, I verify that no checked-in file disappeared
in the process, and lo and behod, it found that the following file
got wiped:
- gdb/testsuite/dg-extract-results.py:
And it's not part of the tarball either.
I don't understand while we delete all *.py files in gdb/testsuite,
since I don't see a rule that expected to create one. A run of the
testsuite also doesn't seem to be creating .py files there.
I traced this to the following commit, which unfortunately provided
no explanation. Perhaps we used to run some tests in the gdb/testsuite
directory and caused files to be left behind there. Perhaps we still
do today?
In the meantime, Executive Decision: In order to allow me to create
tarballs without losing files, I removed it. It's easy to put something
back if we find out why it might still be needed.
gdb/testsuite/ChangeLog:
* Makefile.in (clean mostlyclean): Do not delete *.py.
Tested on x86_64-linux by running the src-release.sh script again,
and this time, dg-extract-results.py no longer gets wiped.
I am completely comfortable with the idea of this change not being
correct, or needing a tweak. It's not a great harm, and I'd rather
get done with the first pre-release, so I went ahead. Just holler
if you see a problem.
Pushed on master + gdb-7.9-branch.
Thanks,
--
Joel
---
gdb/testsuite/ChangeLog | 4 ++++
gdb/testsuite/Makefile.in | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 868339c..4a903bc 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,9 @@
2015-01-13 Joel Brobecker <brobecker@adacore.com>
+ * Makefile.in (clean mostlyclean): Do not delete *.py.
+
+2015-01-13 Joel Brobecker <brobecker@adacore.com>
+
* gdb.python/py-lookup-type.exp: New file.
2015-01-12 Pedro Alves <palves@redhat.com>
diff --git a/gdb/testsuite/Makefile.in b/gdb/testsuite/Makefile.in
index b319409..53cb754 100644
--- a/gdb/testsuite/Makefile.in
+++ b/gdb/testsuite/Makefile.in
@@ -242,7 +242,7 @@ force:;
clean mostlyclean:
-rm -f *~ core *.o a.out xgdb *.x *.grt bigcore.corefile .gdb_history
- -rm -f core.* *.tf *.cl *.py tracecommandsscript copy1.txt zzz-gdbscript
+ -rm -f core.* *.tf *.cl tracecommandsscript copy1.txt zzz-gdbscript
-rm -f *.dwo *.dwp
-rm -rf outputs temp cache
-rm -f read1.so expect-read1
--
1.9.1
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2015-01-13 15:33 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-13 15:33 [pushed/master/gdb-7.9-branch] gdb/testsuite: Make clean mostlyclean should not delete *.py Joel Brobecker
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).