[Patches] [Patch #100803] puremodule: ANSI-fication (UNTESTED!)

noreply@sourceforge.net noreply@sourceforge.net
Wed, 8 Nov 2000 14:57:27 -0800


Patch #100803 has been updated. 

Project: python
Category: Modules
Status: Closed
Summary: puremodule: ANSI-fication (UNTESTED!)

Follow-Ups:

Date: 2000-Jul-10 10:16
By: nowonder

Comment:
cannot test this, maybe Barry can
-------------------------------------------------------

Date: 2000-Jul-31 01:46
By: nowonder

Comment:
should I just check it in? Did you get around to review it?
-------------------------------------------------------

Date: 2000-Jul-31 09:30
By: bwarsaw

Comment:
There's one typo in nowonder's patch.  Here's a fixed version of the patch (unified).  This compiles but doesn't link.  I can't tell if this is caused by a problem with the purify stub libraries.

Meta note: because this is a commercial product which it seems no one has access to anymore, should we continue to support it in the standard distribution?
-------------------------------------------------------

Date: 2000-Jul-31 10:30
By: gvanrossum

Comment:
Since no-one can test this, let's be conservative and not check it in.
There's no absolute requirement to move *everything* to ANSI.
Since we know the old version worked, we should leave it alone.
I vote to keep this in the distribution -- maybe someone else can use it.

If someone else can confirm that it in fact works (or make it work) I'd be happy to accept the patch at that point.
-------------------------------------------------------

Date: 2000-Oct-26 13:51
By: gvanrossum

Comment:
Barry, why don't you check this in. In the next half year, we'll go through enough alpha and beta tests for 2.1 to make sure that someone will test the new module; and if nobody does, it's likely that nobody ever uses it, so then it doesn't matter. :-)
-------------------------------------------------------

Date: 2000-Nov-08 14:57
By: bwarsaw

Comment:
Heh, this was already applied and checked into version 2.4 of puremodule.c.  I guess I forgot to close this patch. :(
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=100803&group_id=5470