[FFmpeg-devel] [RFC] allow overriding of $nm

[Måns Rullgård]

Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:

> Hello,
> some systems like OpenBSD lack a POSIX-conforming nm tool.
> Now obviously I am not motivated to try to get them fixed,

Sadly, neither are the OpenBSD maintainers.

> so one possibility is to allow changing the selected nm tool.
> This can either be used in combination with your own "fixed" nm but
> also just be misused e.g. like this:
> --nm="echo _ff_extern #"
> or in case of OpenBSD just
> --nm=true
> Another possibility would be to allow overriding the extern prefix
> directly.
> So, what are the opinions on attached patch?
>
> Greetings,
> Reimar D?ffinger
>
> Index: configure
> ===================================================================
> --- configure	(revision 16087)
> +++ configure	(working copy)
> @@ -129,6 +129,7 @@
>    echo "  --extra-ldflags=ELDFLAGS add ELDFLAGS to LDFLAGS [$LDFLAGS]"
>    echo "  --extra-libs=ELIBS       add ELIBS [$ELIBS]"
>    echo "  --extra-version=STRING   version string suffix []"
> +  echo "  --nm=NM                  use nm tool [$nm]"

$nm will not be set here.

>    echo "  --build-suffix=SUFFIX    library name suffix []"
>    echo "  --arch=ARCH              select architecture [$arch]"
>    echo "  --cpu=CPU                select the minimum required CPU (affects"
> @@ -919,6 +920,7 @@
>      host_ldflags
>      host_libs
>      logfile
> +    nm
>      source_path
>      target_exec
>      target_os
> @@ -1097,7 +1099,7 @@
>  host_cc_default="gcc"
>  yasmexe="yasm"
>  ar="ar"
> -nm="nm"
> +nm_default="nm"
>  ranlib="ranlib"
>  strip="strip"
>  ln_s="ln -sf"
> @@ -1265,13 +1267,14 @@
>  cc_default="${cross_prefix}${cc_default}"
>  yasmexe="${cross_prefix}${yasmexe}"
>  ar="${cross_prefix}${ar}"
> -nm="${cross_prefix}${nm}"
> +nm_default="${cross_prefix}${nm}"
>  ranlib="${cross_prefix}${ranlib}"
>  strip="${cross_prefix}${strip}"
>
>  set_default cc
>  enabled cross_compile || host_cc_default=$cc
>  set_default host_cc
> +set_default nm

Add 'nm' to the same line as cc instead.  host_cc is separate because
it depends on the final value of cc.

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