[FFmpeg-devel] Patch to configure to allow --as to override of gas-preprocessor.pl for darwin platform

[Måns Rullgård]

Paul Sterne <make4ppc at gmail.com> writes:

> Could the value of --as be saved before it is set to a default?
> Then if --as was specified on the configure command line "
> gas-preprocessor.pl" wouldn't replace it.
>
> 2010/7/16 M?ns Rullg?rd <mans at mansr.com>
>
>> Paul Sterne <make4ppc at gmail.com> writes:
>>
>> > I want to patch configure to allow --as to override gas-preprocessor.plfor
>> > darwin platform.  This will simplify testing of new versions of
>> > gas-preprocessor.pl and also allow ffmpeg to be built with the
>> > gas-preprocessor.pl feature turned off.
>> >
>> > Index: configure
>> > ===================================================================
>> > --- configure (revision 24277)
>> > +++ configure (working copy)
>> > @@ -2226,7 +2226,7 @@
>> >          ;;
>> >      darwin)
>> >          enable malloc_aligned
>> > -        gas="gas-preprocessor.pl $cc"
>> > +        gas=${as:-"gas-preprocessor.pl $cc"}
>>
>> This doesn't work since $as will always be set there.  Overriding it
>> can of course be useful, so I'll try to think of something that works.
>>
>> --
>> M?ns Rullg?rd
>> mans at mansr.com
>
>>
>
> Index: configure
> ===================================================================
> --- configure	(revision 24278)
> +++ configure	(working copy)
> @@ -1923,6 +1923,7 @@
>  : ${as_default:=$cc}
>  : ${dep_cc_default:=$cc}
>  : ${ld_default:=$cc}
> +as_before_default=$as
>  set_default ar as dep_cc ld
>  
>  test -n "$CC_DEPFLAGS" || CCDEP=$DEPEND_CMD
> @@ -2226,7 +2227,7 @@
>          ;;
>      darwin)
>          enable malloc_aligned
> -        gas="gas-preprocessor.pl $cc"
> +        gas=${as_before_default:-"gas-preprocessor.pl $cc"}
>          enabled ppc && add_asflags -force_cpusubtype_ALL
>          SHFLAGS='-dynamiclib -Wl,-single_module -Wl,-install_name,$(SHLIBDIR)/$(SLIBNAME),-current_version,$(LIBVERSION),-compatibility_version,$(LIBMAJOR) -Wl,-read_only_relocs,suppress'
>          strip="${strip} -x"

No way.  Leave it with me.  I'll fix it properly.

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