This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Fix mingw32 failures due to incorrect directory separator in pattern
- From: Yao Qi <yao at codesourcery dot com>
- To: <lgustavo at codesourcery dot com>
- Cc: "'gdb-patches at sourceware dot org'" <gdb-patches at sourceware dot org>
- Date: Tue, 14 Oct 2014 20:31:22 +0800
- Subject: Re: [PATCH] Fix mingw32 failures due to incorrect directory separator in pattern
- Authentication-results: sourceware.org; auth=none
- References: <542ED60B dot 2080700 at codesourcery dot com>
Luis Machado <lgustavo@codesourcery.com> writes:
> Some testcases, mostly gdb.reverse ones, assume the presence of a '/'
> directory separator before the source file name. This is incorrect for
> mingw32 hosts, generating false failures for those tests.
These fails are remote host related, not only specific to mingw32. In
remote host testing, source files can be copied to the place without
directory separator. These fails this patch fixes can be reproduced on
remote linux host with a modified board file
local-remote-host-native.exp too,
$ make check RUNTESTFLAGS="--host_board=local-remote-host-native --target_board=local-remote-host-native py-breakpoint.exp"
FAIL: gdb.python/py-breakpoint.exp: test_bkpt_deletion: continue to breakpoint: Break at multiply.
FAIL: gdb.python/py-breakpoint.exp: test_bkpt_deletion: continue to breakpoint: Break at end.
FAIL: gdb.python/py-breakpoint.exp: test_bkpt_eval_funcs: continue to breakpoint: Break at multiply, i==3
FAIL: gdb.python/py-breakpoint.exp: test_bkpt_eval_funcs: continue to breakpoint: Break at multiply, count==1
FAIL: gdb.python/py-breakpoint.exp: test_bkpt_temporary: continue to breakpoint: Break at multiply.
We hack ${board}_download in local-remote-host-native.exp
proc ${board}_download { board src dest } {
global env
# Copy file to $HOME and return its base name.
set destfile [file tail $src]
file copy -force $src /home/$env(USER)/$destfile
return $destfile
}
>
> I attempted to catch most of the occurrences of the pattern
> ".*/$srcfile" and replaced them with ".*$srcfile". The latter is used
> elsewhere in the testsuite. The resulting patch is attached.
>
> I also see other occurrences of the same assumption throughout the
> testsuite, but usually they are arguments for function calls and i
> seem to recall either the test harness or GDB deals with those paths
> properly.
>
> Is the attached OK?
It is OK.
--
Yao (éå)