[asterisk-r2] libopenr2: -O0 build CFLAGS

[Moises Silva]

Hi Tzafrir,

No reason today, in the early days people built openr2 manually always and
then when they contacted me because of a crash or hang I hated having to
ask them to rebuild with debugging enabled to get usable core dumps or
being able to attach with gdb and poke around the process' memory.

Now days the library is stable and there are distro packages, so no real
need for them anymore. I'll remove them and make a release as soon as I
have a chance. I have one small bug fix [1] in the pipe line too, so it's a
good chance to make a release.

Do you *need* a release at all now? or you can make that change on your end
in the meantime?

Moy

[1]
https://github.com/moises-silva/openr2/commit/4d3df2dcbe04034e85385021a630308358a8fe74

On Sun, Jan 26, 2014 at 11:53 AM, Tzafrir Cohen <tzafrir.cohen at xorcom.com>wrote:

> Hi
>
> The Debian build system wants to set its own build flags. They did not
> get set automatically on openr2 due to the following lines in
> configure.in:
>
> # let's default to compile with debugging information
> # most users will not even care about it
> CFLAGS='-ggdb3 -O0'
> CXXFLAGS='-ggdb3 -O0'
>
> Is there any real point in those? Linux distributions build with -g for
> quite some times (and keep the debug information separately). -O0 is a
> bit odd. I removed those lines in my latest build. Any reason to keep
> them?
>
> --
>                Tzafrir Cohen
> icq#16849755              jabber:tzafrir.cohen at xorcom.com
> +972-50-7952406           mailto:tzafrir.cohen at xorcom.com
> http://www.xorcom.com
>
> --
> _____________________________________________________________________
> -- Bandwidth and Colocation Provided by http://www.api-digital.com --
>
> asterisk-r2 mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-r2
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-r2/attachments/20140127/38602478/attachment.html>