[Pkg-mpd-maintainers] Bug#998310: Bug#998310: mpd: fails to start with "Assertion `sockets.empty()' failed."

[Florian Schlichting]

Hi kaliko,

On Fri, Nov 05, 2021 at 11:52:14AM +0100, kaliko wrote:
> Then I believe setting NDEBUG (either at meson level with "b_ndebug=true" or
> within "DEB_CXXFLAGS_MAINT_APPEND") will lead to the same result as meson
> options "--buildtype=debugoptimized -Db_ndebug=true" since dpkg-buildflags
> already set "-g -O2".
> 
> What do you think of this Florian?

I think that sounds sensible. We want to generate debug info for
dbgsym/gdb, and as Max says he intends the assertions for hunting down
bugs and doesn't expect normal users to run builds that include them,
disabling them is the correct thing to do.

Following your links I found Ian's explanation at
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=711515#30, which
probably explains the thinking behind Debian's defaults, and why this is
a situation where we should deviate from them.

Will you make the change?

Florian