[Python-bugs-list] [ python-Bugs-489052 ] define NDEBUG for release builds

[noreply@sourceforge.net]

Bugs item #489052, was opened at 2001-12-04 12:13
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=489052&group_id=5470

Category: Build
Group: Python 2.2
Status: Closed
Resolution: Fixed
Priority: 7
Submitted By: Tim Peters (tim_one)
Assigned to: Tim Peters (tim_one)
Summary: define NDEBUG for release builds

Initial Comment:
NDEBUG should be defined on the command line during 
release builds of Python, so that assert() statements 
go away.  We tried defining it in Python.h, but that 
interferes with extensions linking Python.h (and 
created real problems for Robin Dunn).

The Windows build already does this.

The Linux build needs it, and assigned to Fred for 
that part.

Fred, please assign to jackjansen when you're done, so 
he can consider the Mac build.

Any other build need special treatment?

----------------------------------------------------------------------

>Comment By: Jack Jansen (jackjansen)
Date: 2001-12-06 13:52

Message:
Logged In: YES 
user_id=45365

Little to be done about this: these files are the equivalent
of unix -D options. If someone comes across an extension for which this is a problem they'll have to roll their one definitions file without the NDEBUG.

----------------------------------------------------------------------

Comment By: Tim Peters (tim_one)
Date: 2001-12-06 12:15

Message:
Logged In: YES 
user_id=31435

Thanks, Jack.  I'm closing this now.  If the BeOS or OS2 
maintainters exist, they're welcome to identify themselves 
<wink>.

BTW, are you sure you wanted to #define NDEBUG in header 
files on the Mac?  The thrust of the bug report was to get 
rid of that very practice on Linux.

----------------------------------------------------------------------

Comment By: Jack Jansen (jackjansen)
Date: 2001-12-05 15:02

Message:
Logged In: YES 
user_id=45365

Fixed for MacPython. Assigning it back to Tim, I don't know whether the other maintainers (like for BeOS or 
OS2) need to be informed of this too.


----------------------------------------------------------------------

Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2001-12-04 12:56

Message:
Logged In: YES 
user_id=3066

The Unix portion of this is fixed in configure.in revision
1.284.  Passing to Jack for Mac OS magic.

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=489052&group_id=5470