#39 [tcl] Fixes for building on Darwin & Tcl stubs support

closed-out-of-date
nobody
None
5
2008-03-26
2003-08-25
No

The attached patch (against CVS HEAD) fixes the following
issues:

- m4 macros not installed into DESTDIR
- Darwin specific LDFLAGS not passed to libtool
- use CFLAGS -fno-common on Darwin to avoid issue
detailed in swig patch #775635
- switched tcl module building to use stubs in all cases,
there is no reason to support building non-stubs tcl
modules nowadays, indeed no common tcl extensions
build without stubs anymore...
in particular, build libswigtcl using stubs as well
and don't link it with libtcl.
- on Darwin, tcl module building needs specific values of
TCLSO & TCLLDSHARED & TCLCXXSHARED
in particular, tcl modules on Darwin are .dylibs
and not .bundles ever since around tcl 8.3.2,
unlike modules for pyton/perl etc.
- use more settings from tclConfig.sh:
TCLINCLUDE, TCLSTUBLIB,
TCLSO, TCLLDSHARED, TCLCXXSHARED
the last three only on Darwin at present, may
work on other platforms as well; in particular
this abstracts away the above Darwin specific issue
with tcl module linking.
- in all tcl examples/tests, load the library using
file extn obtained from [info sharedlibextension]
to abstract away platform differences.

With these changes, swig sucessfully completes all tcl
tests on Darwin built against tcl 8.4.4.

To build swig against Tcl built as a framework (now
standard build style on Darwin, will be included with
OS X 10.3) use:
./configure \ --with-tclconfig=/Library/Frameworks/Tcl.framework

Discussion

  • Daniel A. Steffen

     
  • Daniel A. Steffen

    • assigned_to: nobody --> beazley
     
  • alan johnson

    alan johnson - 2004-11-05

    Logged In: YES
    user_id=943591

    this looks good

     
  • Daniel A. Steffen

    Logged In: YES
    user_id=90580

    I have a more recent patch for these issues, will try dig it up and update it
    to the HEAD in the next few days

     
  • Olly Betts

    Olly Betts - 2006-09-23

    Logged In: YES
    user_id=14972

    I'm sorry this patch has languished so long.

    I've just had a look through it. The patch won't apply
    cleanly now, and I'm not able to test on Darwin, so I'm not
    confident enough to try to apply those parts.

    The "[info sharedlibextension]" changes have already been
    made, except for one file which was missed - I've just fixed
    that.

    das: Would you mind updating and testing any parts of this
    patch which are still relevant, and posting an updated patch
    on the tracker? I'll do my best to make sure it gets dealt
    with more quickly this time!

     
  • Olly Betts

    Olly Betts - 2006-09-25
    • summary: Fixes for Darwin Tcl module building and Tcl stubs support --> [tcl] Fixes for building on Darwin & Tcl stubs support
     
  • Olly Betts

    Olly Betts - 2006-10-01

    Logged In: YES
    user_id=14972

    Just noticed that there are more non-Darwin specific changes
    left. I'm just taking a look at merging in the use of stubs
    for the testsuite. Reassigning since I'm working on this one...

     
  • Olly Betts

    Olly Betts - 2006-10-01
    • assigned_to: beazley --> olly
     
  • Olly Betts

    Olly Betts - 2006-12-04
    • assigned_to: olly --> nobody
     
  • Olly Betts

    Olly Betts - 2006-12-04

    Logged In: YES
    user_id=14972
    Originator: NO

    After further investigation, I think this is beyond what I'm comfortable changing for Tcl without a tested patch from someone with more Tcl knowledge. So I'm unassigning this as I'm no longer working on it.

    If you can provided an updated patch for the remaining issues, I'd be happy to take a look.

     
  • Daniel A. Steffen

    Logged In: YES
    user_id=90580
    Originator: YES

    Olly,
    sorry I have not looked at this patch in a long time (it was current for tcl 8.4.4 and we're now at 8.4.14, i.e. more than 3 years later).
    I will try to find some time to produce an updated patch of the changes that are still necessary but it may take a bit of time.

     
  • William Fulton

    William Fulton - 2008-03-26

    Logged In: YES
    user_id=242951
    Originator: NO

    Closing due to lack of updated patch being supplied. Also most of the patches are for when SWIG used to build the runtime shared objects. SWIG doesn't build them any more, so this is out of date. Feel free to re-open if an updated patch is supplied.

     
  • William Fulton

    William Fulton - 2008-03-26
    • status: open --> closed-out-of-date
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks