[Bug binutils/5146] bfd build fails in alloca-conf.h when built without gcc

[haubi at gentoo dot org]

------- Additional Comments From haubi at gentoo dot org  2007-10-15 09:51 
-------
Hi Nick,

(In reply to comment #1)

> Fixing the bfd configure script so that the system alloca.h is used instead 
> does not seem to me to be sufficient - the problem can still arise in other 
> build directories.

Did not have this problem in other directories - actually, "alloca-conf.h" is
used in bfd only.

But agreed, this is not sufficient for platforms really lacking alloca.h

> So please could you try out the patch below (which fixes 
> the header itself) and let me know if it works for you ?

With this patch only (without checking for <alloca.h> in bfd/configure), the
next unknown symbol is 'size_t':

libtool: compile:  cc -Wp,-H256000 -DHAVE_CONFIG_H -I.
-I/source/binutils-2.18/bfd -I. -DHOST_HPPAHPUX -D_LARGEFILE64_SOURCE
-DHPUX_CORE -I. -I/source/binutils-2.18/bfd
-I/source/binutils-2.18/bfd/../include -g -c /source/binutils-2.18/bfd/som.c -o
som.o
cc: "/source/binutils-2.18/bfd/../include/alloca-conf.h", line 15: error 1506:
Parameters allowed in function definition only.

So in case <alloca.h> is missing, alloca-conf.h should include both ansidecl.h
for PTR (as there is a complex #ifdef for __STDC__) as well as sys/types.h (if
available) for size_t ?

Eh, in gas/ChangeLog-9295 there is this log entry: "Replace alloca-conf.h
inclusion with code recommended in autoconf documentation."

Maybe this should be done for bfd too ?

Thanks!
/haubi/

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=5146

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.