Tcl Source Code

View Ticket
Login
Ticket UUID: 1092952
Title: TEA patch for MSVC7
Type: Patch Version: None
Submitter: jenglish Created on: 2004-12-29 21:39:54
Subsystem: 85. tclconfig Assigned To: hobbs
Priority: 5 Medium Severity:
Status: Closed Last Modified: 2004-12-30 08:48:17
Resolution: Fixed Closed By: hobbs
    Closed on: 2004-12-30 00:10:30
Description:
Patch to sampleextension/tclconfig/tcl.m4 to allow TEA
extensions to build with MSVC7 (the "free" edition of
the VC++ toolkit).

Summary of changes:

+ Don't use "-GD"
+ Don't use "-link50compat"
+ Don't use "-YX"
+ Use "LINK -LIB" instead of "LIB" for STLIB_LD.

Related information: http://wiki.tcl.tk/11431
User Comments: hobbs added on 2004-12-30 08:48:17:
Logged In: YES 
user_id=72656

Applied to the core as well.

hobbs added on 2004-12-30 07:10:30:
Logged In: YES 
user_id=72656

GD, link50compat and YX removed, link -lib used, MD left alone.

jenglish added on 2004-12-30 04:47:54:
Logged In: YES 
user_id=68433

More info:

"/GD" supposedly means "Optimize for Windows DLL", but
according to the manual it's also "Reserved for future use".
 Apparently a no-op in VC6 and earlier; not supported at all
in VC7.

"-link50compat" isn't needed for TEA extensions under normal
circumstances to begin with: it's only useful if the
extension (a) builds an import library (either for the
extension itself or for a stub library) that (b) needs to be
compatible with older Windows compilers.

"-YX" generates a *.PCH file under VC6; it's not supported
under VC7, and is not necessary even under VC6.  VC6 users
who really want the *.pch file can use "CFLAGS=-YX" at
configure-time to reenable this feature.

LIB.EXE does not seem to be included in MSVC7;  LINK /LIB
should work for VC6 and earlier versions as well as VC7.

It might be worthwhile to also remove "/MD" ("link with
MSVCRT"), but I'm not sure of the full ramifications of
this.  Since LIBTCLSTUB.LIB already has a DEFAULTLIB:MSVCRT
dependency, it might not do any good anyway.

jenglish added on 2004-12-30 04:40:50:

File Added - 113833: tea-vc7-compat.patch

Attachments: