This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Your sim testsuite changes broke other targets
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: MR dot Swami dot Reddy at nsc dot com
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 8 Apr 2008 13:16:58 +0200
- Subject: Your sim testsuite changes broke other targets
All tests broke for all targets in my sim autotester. For
*some* reason, they now want to run cr16-elf-as for the tests...
2008-02-12 M Ranga Swami Reddy <MR.Swami.Reddy@nsc.com>
* sim/cr16/: New directory. Tests for CR16 simulator.
sim/cr16/allinsn.exp and sim/cr16/misc.exp: New files: Test scripts.
At the top of that allinsn.exp:
------
# CR16 simulator testsuite.
set target_alias cr16-elf
set global_ld_options "-L/local/user/swami/cr16-5.1/build/CR_R_2007-08-24_11-34-56/release/linux/lib/gcc/cr16-elf/4.1.2 -T/home10/swami/dummy.ld"
if [istarget cr16*-*-*] {
...
------
I think I see why! (Hint: they aren't cr16-elf. ;)
Did you really mean to commit with those "set"s there? Please
remove them; they look wrong. At least move them inside the
"if". (If you remove them, I'll consider adding cr16 to the
targets I test in my autotester.)
brgds, H-P