Public SourceMage Discussion List

[Ismael Luceno <ismael.luceno AT gmail.com>]

On Sun, 10 Mar 2013 18:23:36 -0600
Javier Vasquez <j.e.vasquez.v AT gmail.com> wrote:
> On Sun, Mar 10, 2013 at 6:04 PM, Javier Vasquez
> <j.e.vasquez.v AT gmail.com> wrote:
> > On Sun, Mar 10, 2013 at 5:41 PM, Remko van der Vossen
> > <wich AT yuugen.jp> wrote:
> >> Hi Javier,
> >>
> >> On 2013/03/11, at 2:34, Javier Vasquez <j.e.vasquez.v AT gmail.com>
> >> wrote:
> >>
> >>> ...
> >>
> >> The problem is that the Makefile that adds the attribute removes
> >> the inline keyword itself:
> >>
> >>> -Dinline=__attribute__\ \(\(__always_inline__\)\)
> >>
> >>
> >>> ...
> >>
> >> Just change the makefile to say:
> >>
> >>> -Dinline=inline\ __attribute__\ \(\(__always_inline__\)\)
> >>
> >> Regards, Remko
> >
> >
> > Great !   I'll try it...
> >
> > There are several Makefiles with such line, though this is the 1st
> > one failing, so I guess I have to fix all of them, :-)
> >
> > I'll report back when having results...
> 
> 
> The following additions to PRE_BUILD worked like a charm:
> 
> sedit "s:__attribute__ ((__always_inline__)):inline __attribute__
> ((__always_inline__)):" configure &&
> sedit "s:__attribute__ ((__always_inline__)):inline __attribute__
> ((__always_inline__)):" aclocal.m4 &&
> sedit "s:__attribute__ ((__always_inline__)):inline __attribute__
> ((__always_inline__)):" acinclude.m4
> 

It's probably better to use '__inline__' and avoid adding it multiple
times:
sedit '/__inline/n;
s/__attribute__ ((__always_inline__))/__inline__ &/'

Attachment: signature.asc
Description: PGP signature