[PATCH] PR other/63613: Add fixincludes for dejagnu.h
Rainer Orth
ro@CeBiTec.Uni-Bielefeld.DE
Thu Dec 11 13:07:00 GMT 2014
David Malcolm <dmalcolm@redhat.com> writes:
> On Mon, 2014-12-08 at 14:13 +0100, Rainer Orth wrote:
>> Jeff Law <law@redhat.com> writes:
>>
>> > On 12/04/14 15:42, Rainer Orth wrote:
>> >> David Malcolm <dmalcolm@redhat.com> writes:
>> >>
>> >>> <dejagnu.h> assumed -fgnu89-inline until a recent upstream fix;
>> >>> see http://lists.gnu.org/archive/html/dejagnu/2014-10/msg00011.html
>> >>>
>> >>> Remove the workaround from jit.exp that used -fgnu89-inline
>> >>> in favor of a fixincludes to dejagnu.h that applies the upstream fix
>> >>> to a local copy.
>> >>>
>> >>> This should make it easier to support C++ testcases from jit.exp.
>> >>
>> >> I wonder how this would work if dejagnu.h doesn't live in a system
>> >> include dir (e.g. a self-compiled version)? fixincludes won't touch
>> >> those AFAIU. The previous version with -fgnu89-inline would still work
>> >> in that case provided dejagnu.h is found at all.
>> > Presumably in that case the answer is upgrade dejagnu? :-)
>>
>> I've two problems with this:
>>
>> * There's not yet a DejaGnu release available with the fix and I've no
>> idea if there are any planned any time soon. Not everyone is
>> comfortable with random git (or whatever) snapshots.
>
> FWIW I've asked on the DejaGnu mailing list, and Ben Elliston said:
>> Yes. I plan on releasing 1.6 over the holidays.
> http://lists.gnu.org/archive/html/dejagnu/2014-12/msg00001.html
Thanks for checking this, but ...
>> * I don't consider this a critical issue that cannot work without
>> current releases. We're already working around several upstream
>> DejaGnu issues in our codebase, and I don't consider this particular
>> one important enough to require everyone to upgrade to a not-a-release
>> version.
... a DejaGnu 1.6 release would only address one part of my concern: I
still don't believe this minor issues warrants us demanding all gcc
testers upgrading to a newer DejaGnu release. I'd like my fellow
testsuite maintainers to weigh in, though.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
More information about the Gcc-patches
mailing list