[FFmpeg-devel] [PATCH 2/2] configure: add --fatal-warnings option.

[Clément Bœsch]

On Sun, Nov 11, 2012 at 09:39:10PM +0100, Nicolas George wrote:
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  configure |    8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index 0700b1f..b147a7a 100755
> --- a/configure
> +++ b/configure
> @@ -63,6 +63,7 @@ Options: [defaults in brackets after descriptions]
>  
>  Help options:
>    --help                   print this message
> +  --fatal-warnings         fail if any configure warning is generated

Is it really appropriate to put that in the "help options"?

>    --list-decoders          show all available decoders
>    --list-encoders          show all available encoders
>    --list-hwaccels          show all available hardware accelerators
> @@ -2239,6 +2240,8 @@ for opt do
>      ;;
>      --help|-h) show_help
>      ;;
> +    --fatal-warnings) enable fatal_warnings
> +    ;;
>      *)
>      optname="${opt%%=*}"
>      optname="${optname#--}"
> @@ -4369,7 +4372,10 @@ echo "#endif /* AVUTIL_AVCONFIG_H */" >> $TMPH
>  
>  cp_if_changed $TMPH libavutil/avconfig.h
>  
> -test -n "$WARNINGS" && printf "\n$WARNINGS"
> +if test -n "$WARNINGS"; then
> +  printf "\n$WARNINGS"
> +  enabled fatal_warnings && exit 1

style: 4 spaces preferred

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121114/731c1928/attachment.asc>