[Patches] [Patch #103868] Add better default compiler support

[noreply@sourceforge.net]

Patch #103868 has been updated. 

Project: python
Category: distutils
Status: Accepted
Submitted by: lemburg
Assigned to : akuchling
Summary: Add better default compiler support

Follow-Ups:

Date: 2001-Feb-18 14:47
By: akuchling

Comment:
OK; approved, then.  I'd suggest turning the dictionary 
into a list of 2-tuples, then.

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

Date: 2001-Feb-18 14:28
By: lemburg

Comment:
The RE-match will provide the best forward compatibility and
yes, cygwin's sys.platform does include the OS-type and cygwin
version:  cygwin_nt-5.01, cygwin_98-4.101

Since distutils is an installer and in no way performance sensitive,
I think that flexibility is more important than dictionary lookup
speed.
-------------------------------------------------------

Date: 2001-Feb-18 14:22
By: akuchling

Comment:
Can you convince me why a regular expression match is required, instead of
just normalizing sys.platform 
('cygwin/1.14' -> 'cygwin') and continuing to use a 
dictionary? 

(Does sys.platform actually include a version number on Cygwin?  Wonder if
that's a good idea?)

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

Date: 2001-Feb-18 14:01
By: lemburg

Comment:
Assigned to Andrew for review.
-------------------------------------------------------

Date: 2001-Feb-18 14:00
By: lemburg

Comment:
Assigning to Andrew for review.
-------------------------------------------------------

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

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