Re: [Sofia-sip-devel] NDEBUG enable not working...

2008-07-09 Thread Colin Whittaker




FYI,

I just built 1.12.9, and --enable-ndebug did NOT add -DNDEBUG to the
command line.
It appears the sac-general.m4 still has the 'x' and
enable_expensive_checks still does too.

Colin..


Pekka Pessi wrote:

  2008/6/19 Fabio Margarido [EMAIL PROTECTED]:
  
  
Yes, I've already sent a patch to fix this but it never got in I'm afraid...

  
  
It is accepted but since my time machine is still in service it has
not yet got into the 1.12.6.

--Pekka

  
  
On Thu, Jun 19, 2008 at 5:11 PM, Colin Whittaker
[EMAIL PROTECTED] wrote:


  I'm trying to get rid of the possibility of assert() by passing
--enable_ndebug to configure.
It does not seem to work.
I think the problem may be in m4/sac-general.m4
Starting at line 323.
AC_DEFUN([SAC_ENABLE_NDEBUG],[
AC_REQUIRE([SAC_TOOL_CC])
AC_ARG_ENABLE(ndebug,
[  --enable-ndebug compile with NDEBUG [[disabled]]],
 , enable_ndebug=no)
AM_CONDITIONAL(NDEBUG, test x$enable_ndebug = yes)
])

Shouldn't the test be just $enable_ndebug = yes, without the 'x' ?

Looks like enable_expensive_checks might have the same problem.

Colin..


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel

  

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel


  
  


  




-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel


Re: [Sofia-sip-devel] NDEBUG enable not working...

2008-06-22 Thread Pekka Pessi
2008/6/19 Fabio Margarido [EMAIL PROTECTED]:
 Yes, I've already sent a patch to fix this but it never got in I'm afraid...

It is accepted but since my time machine is still in service it has
not yet got into the 1.12.6.

--Pekka

 On Thu, Jun 19, 2008 at 5:11 PM, Colin Whittaker
 [EMAIL PROTECTED] wrote:
 I'm trying to get rid of the possibility of assert() by passing
 --enable_ndebug to configure.
 It does not seem to work.
 I think the problem may be in m4/sac-general.m4
 Starting at line 323.
 AC_DEFUN([SAC_ENABLE_NDEBUG],[
 AC_REQUIRE([SAC_TOOL_CC])
 AC_ARG_ENABLE(ndebug,
 [  --enable-ndebug compile with NDEBUG [[disabled]]],
  , enable_ndebug=no)
 AM_CONDITIONAL(NDEBUG, test x$enable_ndebug = yes)
 ])

 Shouldn't the test be just $enable_ndebug = yes, without the 'x' ?

 Looks like enable_expensive_checks might have the same problem.

 Colin..


 -
 Check out the new SourceForge.net Marketplace.
 It's the best place to buy or sell services for
 just about anything Open Source.
 http://sourceforge.net/services/buy/index.php
 ___
 Sofia-sip-devel mailing list
 Sofia-sip-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel


 -
 Check out the new SourceForge.net Marketplace.
 It's the best place to buy or sell services for
 just about anything Open Source.
 http://sourceforge.net/services/buy/index.php
 ___
 Sofia-sip-devel mailing list
 Sofia-sip-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel




-- 
Pekka.Pessi mail at nokia.com

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel


Re: [Sofia-sip-devel] NDEBUG enable not working...

2008-06-19 Thread Fabio Margarido
Hi there...

Yes, I've already sent a patch to fix this but it never got in I'm afraid...
I've worked around this by setting CFLAGS manually when I run configure...
Best regards

On Thu, Jun 19, 2008 at 5:11 PM, Colin Whittaker
[EMAIL PROTECTED] wrote:
 I'm trying to get rid of the possibility of assert() by passing
 --enable_ndebug to configure.
 It does not seem to work.
 I think the problem may be in m4/sac-general.m4
 Starting at line 323.
 AC_DEFUN([SAC_ENABLE_NDEBUG],[
 AC_REQUIRE([SAC_TOOL_CC])
 AC_ARG_ENABLE(ndebug,
 [  --enable-ndebug compile with NDEBUG [[disabled]]],
  , enable_ndebug=no)
 AM_CONDITIONAL(NDEBUG, test x$enable_ndebug = yes)
 ])

 Shouldn't the test be just $enable_ndebug = yes, without the 'x' ?

 Looks like enable_expensive_checks might have the same problem.

 Colin..


 -
 Check out the new SourceForge.net Marketplace.
 It's the best place to buy or sell services for
 just about anything Open Source.
 http://sourceforge.net/services/buy/index.php
 ___
 Sofia-sip-devel mailing list
 Sofia-sip-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel