[vlc-devel] [PATCH] configure: enable debug by default

[Jean-Baptiste Kempf]

Yes, +1.

On Thu, 7 Feb 2019, at 13:23, Thomas Guillem wrote:
> It seems that lot of developers forget to enable this option. This option
> enables assert and other debug codes (like the very useful thread/mutex debug
> code) that should be mandatory when you dev on VLC.
> 
> This is quite a big change: all VLC maintainers should now add
> "--disable-debug" when they release a stable version of VLC.
> ---
>  configure.ac | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 9453dd852b..4f4b6b395a 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -979,9 +979,9 @@ dnl
>  dnl  Debugging mode
>  dnl
>  AC_ARG_ENABLE([debug],
> -  AS_HELP_STRING([--enable-debug],
> -    [build with run-time assertions (default disabled)]),,
> -  [enable_debug="no"])
> +  AS_HELP_STRING([--disable-debug],
> +    [build with run-time assertions (default enabled)]),,
> +  [enable_debug="yes"])
>  AH_TEMPLATE(NDEBUG,
>              [Define to 1 if debug code should NOT be compiled])
>  AS_IF([test "${enable_debug}" != "no"], [
> -- 
> 2.20.1
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


-- 
Jean-Baptiste Kempf -  President
+33 672 704 734