[FFmpeg-devel] [PATCH] reuse extern_prefix for YASMFLAGS -DPREFIX

[Måns Rullgård]

Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:

> Hello,
> instead of the ugly hardcode, this sets yasm -DPREFIX when we detected something
> for extern_prefix.
> Actually this only works if extern_prefix is either not set at all or _,
> anything else is currently not supported by the YASM code (though it would
> be easy to extend).
> Index: configure
> ===================================================================
> --- configure	(revision 20477)
> +++ configure	(working copy)
> @@ -2232,10 +2232,10 @@
>      YASMFLAGS="-f $objformat -DARCH_$(toupper $subarch)"
>      enabled     x86_64        && append YASMFLAGS "-m amd64"
>      enabled     pic           && append YASMFLAGS "-DPIC"
> +    test -n "$extern_prefix"  && append YASMFLAGS "-DPREFIX"
>      case "$objformat" in
>          elf) enabled debug && append YASMFLAGS "-g dwarf2" ;;
> -        macho64)              append YASMFLAGS "-DPIC -DPREFIX" ;;
> -        *)                    append YASMFLAGS "-DPREFIX"  ;;
> +        macho64)              append YASMFLAGS "-DPIC" ;;
>      esac
>      disabled yasm || { check_yasm "pabsw xmm0, xmm0" && enable yasm; }
>

Looks reasonable.  Commit if tested.

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