On Thu, Mar 03, 2005 at 10:54:43AM +0100, Jérôme Marant wrote:
> Quoting Kurt Roeckx <[EMAIL PROTECTED]>:
> 
> >Hi,
> >
> >Is there any progress on this?  Did you send that patch to define
> >HAVE_ALLOCA upstream?  I really think that that is all that is
> >missing to get a working version.
> 
> Upstream didn't reply. There is no such HAVE_ALLOCA in the upstream
> code for amd64, because it seems to be autodetected by the configure
> script.
> I'd really like to know why it seems to work upstream and no on
> Debian system. Would you please investigate?

During configure, I see this:
checking for alloca... yes

I also find those:
./mac/inc/config.h:/* #undef HAVE_ALLOCA_H */
./debian/build-x/src/config.h:#define HAVE_ALLOCA_H 1
./debian/build-nox/src/config.h:#define HAVE_ALLOCA_H 1

I added this line to src/config.in:
#undef HAVE_ALLOCA

And it build perfectly then.

I suggest adding C_ALLOCA there too since the autoconf macro
AC_FUNC_ALLOCA can set both.

I have no idea what upstream changed since the version in debian,
they might have put that in src/config.in too and removed the
defines from all other places.

I think the easier solution is just to define it in
./src/m/amdx86-64.h for now.


Kurt


Reply via email to