[FFmpeg-cvslog] r23953 - trunk/configure

[Måns Rullgård]

Ramiro Polla <ramiro.polla at gmail.com> writes:

> Hi,
>
> On Thu, Jul 1, 2010 at 8:21 PM, mru <subversion at mplayerhq.hu> wrote:
>> Author: mru
>> Date: Fri Jul ?2 01:21:30 2010
>> New Revision: 23953
>>
>> Log:
>> Check for GNU assembler, use gas-preprocessor on darwin
>>
>> Modified:
>> ? trunk/configure
>>
>> Modified: trunk/configure
>> ==============================================================================
>> --- trunk/configure ? ? Fri Jul ?2 01:21:27 2010 ? ? ? ?(r23952)
>> +++ trunk/configure ? ? Fri Jul ?2 01:21:30 2010 ? ? ? ?(r23953)
>> @@ -988,6 +988,7 @@ HAVE_LIST="
>> ? ? GetProcessMemoryInfo
>> ? ? GetProcessTimes
>> ? ? getrusage
>> + ? ?gnu_as
>> ? ? struct_rusage_ru_maxrss
>> ? ? ibm_asm
>> ? ? inet_aton
>> @@ -2116,6 +2117,7 @@ case $target_os in
>> ? ? ? ? ;;
>> ? ? darwin)
>> ? ? ? ? enable malloc_aligned
>> + ? ? ? ?as="gas-preprocessor.pl $cc"
>
> Shouldn't this be set only for arm/ppc?

Anything using GNU syntax needs it.  We so happen to not have any for x86.
Setting it on x86 does no harm.

>> ? ? ? ? SHFLAGS='-dynamiclib -Wl,-single_module -Wl,-install_name,$(SHLIBDIR)/$(SLIBNAME),-current_version,$(LIBVERSION),-compatibility_version,$(LIBMAJOR) -Wl,-read_only_relocs,suppress'
>> ? ? ? ? strip="${strip} -x"
>> ? ? ? ? FFLDFLAGS="-Wl,-dynamic,-search_paths_first"
>> @@ -2306,6 +2308,9 @@ unsigned int endian = 'B' << 24 | 'I' <<
>> ?EOF
>> ?od -t x1 $TMPO | grep -q '42 *49 *47 *45' && enable bigendian
>>
>> +check_asm gnu_as '".macro m n\n\\n:.int 0\n.endm\nm x"' ||
>> + ? ?warn "GNU assembler not found, install gas-preprocessor"
>> +
>
> And this could probably be turned into an error (like we plan on doing
> for yasm).

This only affects audio codecs (at the moment) so it's less critical.
Besides, the OSX/PPC is quickly fading into obscurity if it's not
there already, so I'd rather not spend too much effort on it.

-- 
M?ns Rullg?rd
mans at mansr.com