From: <no...@so...> - 2002-07-29 01:32:00
|
Bugs item #529801, was opened at 2002-03-14 01:43 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=529801&group_id=10894 Category: 49. Configure and Build Tools Group: 8.4b2 Status: Open Resolution: Fixed Priority: 5 Submitted By: Jean-Claude Wippler (jcw) Assigned to: Daniel A. Steffen (das) Summary: macosx forgets to do ranlib Initial Comment: On the SF compile farm. MacOS X 10.1, running the Tcl make fails because a ranlib call is not being done before ld. It can be resolved by inserting the following lines at the end of the ${TCL_LIB_FILE} target: if test "x@DL_OBJS@" = "xtclLoadDyld.o"; then $(RANLIB) ${TCL_LIB_FILE}; fi ---------------------------------------------------------------------- >Comment By: Mo DeJong (mdejong) Date: 2002-07-28 18:31 Message: Logged In: YES user_id=90858 While the approach in question may have worked for you, it broke the build for other systems. Besides, it just did not make any sense. What on earth does the implementation of dynamic loading have to do with whether ranlib is run on a generated .a archive? Even if those two things were related, the decision of when to run ranlib must be made in the configure script, not the Makefile. You mention that ranlib is not getting run on OS X. I assume you mean at build and not install time, right? The right way to address this is to take a peek at the SC_CONFIG_CFLAGS macro and see what MAKE_LIB is getting set to. I assume that the AC_PROG_RANLIB is not finding ranlib and this is causing MAKE_LIB to not include a ranlib call. Is that the case? ---------------------------------------------------------------------- Comment By: Jean-Claude Wippler (jcw) Date: 2002-07-28 04:53 Message: Logged In: YES user_id=1983 Except that now, on OS X, --disable-shared doesn't build at all. I don't mind having my suggestion ignored or reverted, all I can say is that what I originally submitted worked for me, and that we're not getting very far this way :( Mo, I'll rephrase the problem, in the hope that it works better for you this way: on OS X, a "--disable-shared" (i.e. static) build fails, because a ranlib call is being left out (in two places in the makefile, in fact). ---------------------------------------------------------------------- Comment By: Mo DeJong (mdejong) Date: 2002-07-27 20:29 Message: Logged In: YES user_id=90858 This change along with a couple of others from the Mac OS X branch have been reverted as of 2002-07-27. They were just wrong and they broke the build on other systems. I have gone ahead and created a new set of build and install commands and defined them in the SC_CONFIG_CFLAGS macro in tcl.m4 (see MAKE_LIB, MAKE_STUB_LIB, INSTALL_LIB, and INSTALL_STUB_LIB). If you are still having problems with ranlib related issues related to OS X, please take a look at the new approach and assign any related patches to me. Just an FYI, I don't see how the DL_OBJS in use could have anything to do with ranlib, so please don't go down that route again. ---------------------------------------------------------------------- Comment By: Daniel A. Steffen (das) Date: 2002-07-24 06:55 Message: Logged In: YES user_id=90580 commited to HEAD & core-8-3-1-branch ---------------------------------------------------------------------- Comment By: Daniel A. Steffen (das) Date: 2002-07-24 02:35 Message: Logged In: YES user_id=90580 attached patch to core-8-3-1-branch ---------------------------------------------------------------------- Comment By: Daniel A. Steffen (das) Date: 2002-07-24 02:34 Message: Logged In: YES user_id=90580 attached patch to HEAD ---------------------------------------------------------------------- Comment By: Daniel A. Steffen (das) Date: 2002-07-24 02:13 Message: Logged In: YES user_id=90580 this fix is incorrect, it breaks the shared build as ranlib fails on dynamic libraries: ranlib: file: libtcl8.4.dylib is not an archive to correctly fix static linking (i.e. configured with --disable-shared), IMHO it's best to change the definition of MAKE_LIB in configure.in as in the attached patch (which also reverses the present incorrect change) I'll check this in if Vince agrees ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=529801&group_id=10894 |