Tcl Source Code

View Ticket
Login
Ticket UUID: 794741
Title: sampleextension stublib ranlib fix
Type: Patch Version: None
Submitter: nobody Created on: 2003-08-25 14:59:19
Subsystem: 70. Sample Extension Assigned To: das
Priority: 5 Medium Severity:
Status: Deleted Last Modified: 2005-02-04 19:32:27
Resolution: Fixed Closed By: das
    Closed on: 2005-02-04 12:32:27
Description:
the attached patch fixes an issue with ranlib not being run 
on stub libs when building dynamically linked extensions 
(i.e. with RANLIB=.), this breaks building of 
interdependent stubified extensions on platforms that need 
ranlib to be run on static libs prior to linking (such as Mac 
OS X).

Essentially, ranlib calls on always static libs e.g. stub libs 
need to be separated from ranlib calls on libs that can be 
either static or dynamic, this is done via the new 
RANLIB_STUB which is set to the value of RANLIB before 
RANLIB is reset to . when building a dynamically linked 
extension.

Also added a section to the Makefile illustrating how to 
build stub libraries to clarify usage of the new 
RANLIB_STUB.
The technique adopted at install time is to run 
RANLIB_STUB on libraries containing the string "stub" and 
RANLIB on others.

This issue affects a number of TEA extensions using their 
own stub libs: incrTcl, trf, tkimg, tclxml, tcldom; but 
should also be fixed in sampleextension so that newly 
TEAfied extensions won't perpetuate this problem...

See the following patches for details on how the 
abovementioned extensions are affected:
http://tcltkaqua.sf.net/8.4.4/patches/incrTcl.diff
http://tcltkaqua.sf.net/8.4.4/patches/trf.diff
http://tcltkaqua.sf.net/8.4.4/patches/tkimg.diff
http://tcltkaqua.sf.net/8.4.4/patches/tclxml.diff
http://tcltkaqua.sf.net/8.4.4/patches/tcldom.diff
these will be submitted to the respective projects as time 
permits...
User Comments: das added on 2005-02-04 19:32:27:
Logged In: YES 
user_id=90580

indeed, very old bug, long since fixed

hobbs added on 2005-02-02 07:45:35:
Logged In: YES 
user_id=72656

I believe this is in the sample extension now, correct?

das added on 2003-08-25 22:21:22:

File Added - 59734: sampleextension.diff

das added on 2003-08-25 22:21:21:
Logged In: YES 
user_id=90580

and the patch didn't get attached either...

das added on 2003-08-25 22:18:16:
Logged In: YES 
user_id=90580

oops, looks the browser forgot my login cookie...
submitted by 'das'

Attachments: