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

Thomas Guillem thomas at gllm.fr
Wed Mar 28 15:27:13 CEST 2018


Asserts enabled by default seems to be the new way to go: https://www.gnu.org/software/autoconf/manual/autoconf-2.67/html_node/Particular-Headers.html

"Assertions are enabled by default, but the user can override this by invoking configure with the --disable-assert option."

On Wed, Mar 28, 2018, at 15:13, Rémi Denis-Courmont wrote:
> Le 28 mars 2018 15:39:27 GMT+03:00, "Hugo Beauzée-Luyssen" 
> <hugo at beauzee.fr> a écrit :
> >On Wed, Mar 28, 2018, at 2:35 PM, Thomas Guillem wrote:
> >> For 4.0, I propose to finally do the opposite:
> >>  - Enable asserts by default (NDEBUG is not defined).
> >>  - Disable asserts with --disable-debug for release builds (NDEBUG is
> >defined).
> >> 
> >> Why ? Because too many developers are not using this option that
> >should be
> >> mandatory during the development or testing phase.
> >> ---
> >>  configure.ac | 6 +++---
> >>  1 file changed, 3 insertions(+), 3 deletions(-)
> >> 
> >> diff --git a/configure.ac b/configure.ac
> >> index e1a4800ccb..e1f1e8c748 100644
> >> --- a/configure.ac
> >> +++ b/configure.ac
> >> @@ -952,9 +952,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],
> >> +    [disable 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.11.0
> >> 
> >> _______________________________________________
> >> vlc-devel mailing list
> >> To unsubscribe or modify your subscription options:
> >> https://mailman.videolan.org/listinfo/vlc-devel
> >
> >That seems fine and sane to me. We just want to make sure that various
> >build.sh scripts are passing --disable-debug when building for releases
> >
> >-- 
> >  Hugo Beauzée-Luyssen
> >  hugo at beauzee.fr
> >_______________________________________________
> >vlc-devel mailing list
> >To unsubscribe or modify your subscription options:
> >https://mailman.videolan.org/listinfo/vlc-devel
> 
> It is just moving the problem. Either we screw the clueless developper 
> or we screw the clueless packager.
> -- 
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser 
> ma brièveté.
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list