diff mbox

[committed,libjava] XFAIL sourcelocation (PR libgcj/55637) backported to 4.8.3

Message ID 20140401102903.79BF3FF@mailhost.lps.ens.fr
State New
Headers show

Commit Message

Dominique d'Humières April 1, 2014, 10:29 a.m. UTC
ro@CeBiTec.Uni-Bielefeld.DE (Rainer Orth) wrote:
> Sure, patch preapproved.

Commited as r208983:

2014-04-01  Dominique d'Humieres <dominiq@lps.ens.fr>    
            Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

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


Dominique

Comments

Rainer Orth April 2, 2014, 10:47 a.m. UTC | #1
dominiq@lps.ens.fr (Dominique Dhumieres) writes:

> ro@CeBiTec.Uni-Bielefeld.DE (Rainer Orth) wrote:
>> Sure, patch preapproved.
>
> Commited as r208983:
>
> 2014-04-01  Dominique d'Humieres <dominiq@lps.ens.fr>    
>             Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
>
>         PR libgcj/55637        
>         * testsuite/libjava.lang/sourcelocation.xfail: New file.

Btw, the customary format for such a ChangeLog entry is

2014-04-01  Dominique d'Humieres <dominiq@lps.ens.fr>    

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

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

This way, you can easily see when the original went in.

	Rainer
Dominique d'Humières April 3, 2014, 3:31 p.m. UTC | #2
Thanks for the tip. What should I do now? Should I fix the ChangeLog entry and add a new one or do nothing?

Dominique

Le 2 avr. 2014 à 12:47, Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> a écrit :

> dominiq@lps.ens.fr (Dominique Dhumieres) writes:
> 
>> ro@CeBiTec.Uni-Bielefeld.DE (Rainer Orth) wrote:
>>> Sure, patch preapproved.
>> 
>> Commited as r208983:
>> 
>> 2014-04-01  Dominique d'Humieres <dominiq@lps.ens.fr>    
>>            Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
>> 
>>        PR libgcj/55637        
>>        * testsuite/libjava.lang/sourcelocation.xfail: New file.
> 
> Btw, the customary format for such a ChangeLog entry is
> 
> 2014-04-01  Dominique d'Humieres <dominiq@lps.ens.fr>    
> 
> 	Backport from mainline
> 	2014-02-20  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
> 
>        PR libgcj/55637
>        * testsuite/libjava.lang/sourcelocation.xfail: New file.
> 
> This way, you can easily see when the original went in.
> 
> 	Rainer
> 
> -- 
> -----------------------------------------------------------------------------
> Rainer Orth, Center for Biotechnology, Bielefeld University
Rainer Orth April 3, 2014, 3:33 p.m. UTC | #3
Hi Dominique,

> Thanks for the tip. What should I do now? Should I fix the ChangeLog entry
> and add a new one or do nothing?

if you want, you could fix the ChangeLog entry in place, but don't add a
new one for that change.

	Rainer
Dominique d'Humières April 4, 2014, 7:08 a.m. UTC | #4
> if you want, you could fix the ChangeLog entry in place, but don't add a
> new one for that change.

Done,

Dominique
diff mbox

Patch

Index: libjava/testsuite/libjava.lang/sourcelocation.xfail
===================================================================
--- libjava/testsuite/libjava.lang/sourcelocation.xfail	(revision 0)
+++ libjava/testsuite/libjava.lang/sourcelocation.xfail	(working copy)
@@ -0,0 +1 @@ 
+xfail-output