diff mbox

[libjava] XFAIL sourcelocation (PR libgcj/55637)

Message ID ydda9dm7zwb.fsf@lokon.CeBiTec.Uni-Bielefeld.DE
State New
Headers show

Commit Message

Rainer Orth Feb. 20, 2014, 12:05 p.m. UTC 
The libjava sourcelocation output test has been FAILing on mainline for
more than a year, with no sign of anything happening to resolve that.
To reduce testsuite noise, I suggest to XFAIL the test.  In all mainline
test results from february, only a single set of testresults that
included libjava results (for ia64-suse-linux-gnu) didn't show those
failures.

Tested with the appropriate runtest invocations on i386-pc-solaris2.11
and x86_64-unknown-linux-gnu.

Ok for mainline?

	Rainer


2014-02-20  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	PR libgcj/55637
	* testsuite/libjava.lang/sourcelocation.xfail: New file.

Comments

Bryce McKinlay Feb. 20, 2014, 1:49 p.m. UTC | #1 
On Thu, Feb 20, 2014 at 12:05 PM, Rainer Orth
<ro@cebitec.uni-bielefeld.de> wrote:

> The libjava sourcelocation output test has been FAILing on mainline for
> more than a year, with no sign of anything happening to resolve that.
> To reduce testsuite noise, I suggest to XFAIL the test.  In all mainline
> test results from february, only a single set of testresults that
> included libjava results (for ia64-suse-linux-gnu) didn't show those
> failures.
>
> Tested with the appropriate runtest invocations on i386-pc-solaris2.11
> and x86_64-unknown-linux-gnu.
>
> Ok for mainline?

OK.

The fix for this is to have libjava use libbackttrace to get source
line numbers, rather than rely on external addr2line.

Bryce
Rainer Orth Feb. 20, 2014, 2:06 p.m. UTC | #2 
Bryce McKinlay <bmckinlay@gmail.com> writes:

> The fix for this is to have libjava use libbackttrace to get source
> line numbers, rather than rely on external addr2line.

I see.  Doesn't sound exactly like 4.9 material, though.

Thanks.
        Rainer
diff mbox

Patch

# HG changeset patch
# Parent 6fd38b9012dee330d1343765cfe3cd0e06f33d33
XFAIL sourcelocation (PR libgcj/55637)

diff --git a/libjava/testsuite/libjava.lang/sourcelocation.xfail b/libjava/testsuite/libjava.lang/sourcelocation.xfail
new file mode 100644
--- /dev/null
+++ b/libjava/testsuite/libjava.lang/sourcelocation.xfail
@@ -0,0 +1,1 @@ 
+xfail-output