On 30 Mai, Akos Maroy wrote:

>> Can you please test the CVS version of LAME with the attached patch
>> (I've no gcc 3.1 at the moment)?
> 
> Some more details.
> 
> There is a bug in your GCC_version sed script. Instead of:
> 
> GCC_version=`${CC} --version | sed -n '1s/^[^ ]* (.*) //;s/ .*$//;1p`
> 
> a correct one is:
> 
> GCC_version=`${CC} --version | sed -n '1s/^[[^ ]]* (.*) //;s/ .*$//;1p'`
> 
> 
> The case for the unsupported GCC version is in error.

Fixed, thanks.

> Also, the macro target_cpu is only defined if the AC_CANONICAL_TARGET 
> macro is used in configure.in. As this has just been replaced with 
> AC_CANONICAL_HOST, target_cpu is undefined. I suggest replacing 
> target_cpu in configure.in with host_cpu (which is defined when 
> AC_CANONICAL_HOST is).

Also fixed.

Now I have another problem: the configure script uses
"--no-verify" for libtool. Therefore I _have_ to use
"configure mycpu-blafasel-mysystem" to configure it. As a workaround
I've committed a configure script with --no-verify removed. I haven't
found a way to do it with autoconf. Do you have an idea?

Bye,
Alexander.

-- 
            Secret hacker rule #11: hackers read manuals.

http://www.Leidinger.net                       Alexander @ Leidinger.net
  GPG fingerprint = C518 BC70 E67F 143F BE91  3365 79E2 9C60 B006 3FE7

_______________________________________________
mp3encoder mailing list
[EMAIL PROTECTED]
http://minnie.tuhs.org/mailman/listinfo/mp3encoder

Reply via email to