Alexander Leidinger wrote:
> I fixed this already locally in a similar manner.
Cool
> That's wrong. lame.h doesn't need stdint.h. The correct way of doing
> this is to add
> #ifdef HAVE_STDIINT_H
> #include <stdint.h>
> #endif
> into .c files which actually use *int*_t.
>
> I fix this later this week and commit everything then.
Sorry. I saw in the first instance that this issue came up, the only
file included was lame.h. I'm glad this'll get fixed as well.
> No. But it seems auto* behaves like this. :-(
I took a look into the auto* documentation, and it seems that the
AC_CANONICAL_SYSTEM
macro is responsible for this. This is a deprecated macro (as per the
autoconf info pages), and one should use macros
AC_CANONICAL_BUILD
AC_CANONICAL_HOST
AC_CANONICAL_TARGET
instead. It seems that AC_CANONICAL_TARGET is the one actually to blame
here.
BTW, what is the purpose of this macro? I see there are some safeguards
for cross compilation:
if test ${cross_compiling} = "no"; then
at line 88 and
if test ${cross_compiling} = "yes"; then
at line 154 in configure.in. But cross_compiling is not defined
anywhere, thus none if these tests are true ever.
Maybe this whole AC_CANONICAL_* stuff could be removed?
Akos
_______________________________________________
mp3encoder mailing list
[EMAIL PROTECTED]
http://minnie.tuhs.org/mailman/listinfo/mp3encoder
