This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix for PR59585
- From: Yury Gribov <y dot gribov at samsung dot com>
- To: Yuri Gribov <tetra2005 at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Marek Polacek <polacek at redhat dot com>, Viacheslav Garbuzov <v dot garbuzov at samsung dot com>
- Date: Tue, 24 Dec 2013 10:05:37 +0400
- Subject: Re: [PATCH] Fix for PR59585
- Authentication-results: sourceware.org; auth=none
- References: <52B85DF3 dot 2080705 at samsung dot com> <20131223164035 dot GR892 at tucnak dot redhat dot com> <CAJOtW+6TiLczo6YjadVFV6KWesn+caqKb7-1iHDvWAacZkWNyQ at mail dot gmail dot com>
Yury wrote:
>> Still sounds like a bug elsewhere to me.
>
> Let me investigate this deeper tomorrow (rebuilding fresh Dg, etc.).
So I've double-checked that this is a problem with trunk DejaGNU rsh.exp
script removing trailing newline from test output:
# Delete one trailing \n because that is what `exec' will do and we
want
# to behave identical to it.
regsub "\n$" $output "" output
I can report this to DejaGNU mailing list but even if they agree to fix
we'll still have to do something about legacy Dg installations. I
suggest to work around by removing trailing newline as suggested by
original patch (or maybe replacing it with $ ?).
What's your opinion?
-Y