[FFmpeg-devel] [PATCH] allow overriding SAMPLES with environment variable

[Benoit Fouet]

On Sat, 2 Oct 2010 11:37:11 +0200 Reimar D?ffinger wrote:
> Hello,
> it would be enough to just have it apply if --samples was not passed to
> configure, but I didn't know a good way to do that.
> Currently, you can't just set a SAMPLES environment variable and
> the fate tests will use it, you have to either specify the path
> to configure or specify it on the command-line.
> Both seem a bit inconvenient to me, so I did the following change:
> Index: configure
> ===================================================================
> --- configure   (revision 25309)
> +++ configure   (working copy)
> @@ -3188,7 +3188,9 @@
>  SLIB_EXTRA_CMD=${SLIB_EXTRA_CMD}
>  SLIB_INSTALL_EXTRA_CMD=${SLIB_INSTALL_EXTRA_CMD}
>  SLIB_UNINSTALL_EXTRA_CMD=${SLIB_UNINSTALL_EXTRA_CMD}
> +ifndef SAMPLES
>  SAMPLES=$samples
> +endif

Wouldn't a:
SAMPLES?=$samples

be enough?

Ben