Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-12 Thread Mike Stump
On Dec 11, 2014, at 5:07 AM, Rainer Orth wrote: > David Malcolm writes: >>> * 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 im

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-11 Thread Rainer Orth
David Malcolm writes: > On Mon, 2014-12-08 at 14:13 +0100, Rainer Orth wrote: >> Jeff Law writes: >> >> > On 12/04/14 15:42, Rainer Orth wrote: >> >> David Malcolm writes: >> >> >> >>> assumed -fgnu89-inline until a recent upstream fix; >> >>> see http://lists.gnu.org/archive/html/dejagnu/201

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-09 Thread David Malcolm
On Mon, 2014-12-08 at 14:13 +0100, Rainer Orth wrote: > Jeff Law writes: > > > On 12/04/14 15:42, Rainer Orth wrote: > >> David Malcolm writes: > >> > >>> assumed -fgnu89-inline until a recent upstream fix; > >>> see http://lists.gnu.org/archive/html/dejagnu/2014-10/msg00011.html > >>> > >>> Re

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-08 Thread Rainer Orth
Jeff Law writes: > On 12/04/14 15:42, Rainer Orth wrote: >> David Malcolm writes: >> >>> 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 o

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-06 Thread Bruce Korb
On 12/05/14 12:32, Jeff Law wrote:1 fixincludes/ChangeLog: PR other/63613 * inclhack.def (dejagnu_h_make_inline_functions_static): New fix. * fixincl.x: Regenerate. * tests/base/dejagnu.h: New. OK. No, actually not. +fix = { +hackname = dejagnu_h_make_inline_functions_stat

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-05 Thread Bruce Korb
>> This is the first time I've touched the "fixincludes" directory; >> is this the correct way to make a change here? Well, I'd like to see it -- especially since it's your first. Please send to this gmail account or wait until I get my GNU email this weekend. Thanks! >> Successfully bootstrapped

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-05 Thread Jeff Law
On 12/04/14 15:19, David Malcolm wrote: 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 l

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-05 Thread Jeff Law
On 12/04/14 15:42, Rainer Orth wrote: David Malcolm writes: 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

Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-04 Thread Rainer Orth
David Malcolm writes: > 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 co

[PATCH] PR other/63613: Add fixincludes for dejagnu.h

2014-12-04 Thread David Malcolm
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