On Sat, Aug 09, 2014 at 05:30:25PM +0200, Janne Grunau wrote:
> On 2014-08-09 08:06:12 -0700, Diego Biurrun wrote:
> > ---
> > 
> > Implemented Janne's suggestion.  Untested since I don't have the right 
> > system.
> > 
> >  configure | 15 ++++++++++++---
> >  1 file changed, 12 insertions(+), 3 deletions(-)
> > 
> > --- a/configure
> > +++ b/configure
> > @@ -2199,8 +2199,6 @@ ranlib="ranlib"
> >  yasmexe="yasm"
> >  
> > -nogas=":"
> > -
> >  # machine
> >  arch_default=$(uname -m)
> > @@ -3420,7 +3418,6 @@ case $target_os in
> >          ;;
> >      darwin)
> > -        gas="gas-preprocessor.pl $cc"
> >          enabled ppc && add_asflags -force_cpusubtype_ALL
> >          enabled x86_32 && append SHFLAGS -Wl,-read_only_relocs,suppress
> > @@ -3779,9 +3776,21 @@ EOF
> >  
> >  if enabled_any arm aarch64 || enabled_all ppc altivec && enabled asm; then
> > +    nogas=:
> >      enabled_any arm aarch64 && nogas=die
> >      enabled_all ppc altivec && nogas=warn
> >  
> > +    case $as_type
> > +        arm*) gaspp_as_type=armasm ;;
> > +        gcc)  gaspp_as_type=gas ;;
> > +        *)    gaspp_as_type=$as_type ;;
> > +    esac
> > +
> > +    [ $target_os = "darwin" ] && gaspp_as_type="apple-$as_type"
> > +
> > +    check_cmd gas-preprocessor.pl -arch $arch -as-type $gaspp_as_type &&
> > +        gas="gas-preprocessor.pl -arch $arch -as-type $gaspp_as_type $as"
> 
> missing '--' to divide gaspp options from the assembler cmd line. looks 
> good otherwise. I'll test aarch64/arm with Xcode on darwin and llvm on 
> linux. Martin can hopefully test armasm.

Please push with that amended if it works.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to