octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Patch] Shared build on OS X


From: Per Persson
Subject: Re: [Patch] Shared build on OS X
Date: Sun, 15 Feb 2004 04:42:47 +0100


On Feb 15, 2004, at 01:08, John W. Eaton wrote:

| Octave (CVS as of 2004-02-11) compiles and runs testcases without
| errors (# of expected passes 1199)
| on Mac OS X 10.3.3 using
| gcc 3.3 20030304 (Apple build 1495) and
| g77 3.3.2

Forgot:
./configure --enable-shared --enable-dl --without-hdf5


I applied your patch and make a few additional changes (the new
variables needed to be added to the octave_config_info struct, used in
mkoctfile.in, etc.)

Now mkoctfile uses DL_LD and DL_LDFLAGS and the old
MKOCTFILE_SH_LDFLAGS variable has been replaced by
MKOCTFILE_DL_DLFLAGS.

I think I have the change right, but I don't have a system to use for
testing where there would be a difference between SH_LD and DL_LD, so
you'll need to update your sources and confirm that it works properly.


Good news:
Octave passed all tests
Hello.oct works

Buglets:
* Have to pass -L/opt/local/lib (was specified in LDFLAGS) to mkoctfile for it to find lg2c (FLIBS)

* I forgot to add install names to octave libs (will fix ASAP, set DYLD_LIBRARY_PATH until then)

* Small typo in Makeconf.in
Index: Makeconf.in
===================================================================
RCS file: /cvs/octave/Makeconf.in,v
retrieving revision 1.150
diff -u -d -b -w -r1.150 Makeconf.in
--- Makeconf.in 2004/02/15 00:32:30     1.150
+++ Makeconf.in 2004/02/15 03:18:50
@@ -312,7 +312,7 @@
 # The -L flags to use for the mkoctfile scrip (for --link-stand-alone)
 MKOCTFILE_LFLAGS = -L$(octlibdir)

-MKOCTFILE_DL_LDFLAGS = @MKOCTFILE_DL_DFLAGS@
+MKOCTFILE_DL_LDFLAGS = @MKOCTFILE_DL_LDFLAGS@

 NO_OCT_FILE_STRIP = @NO_OCT_FILE_STRIP@

Nevertheless, quite an improvement.
Compiling octave-forge will probably expose the remaining issues.

/Per


reply via email to

[Prev in Thread] Current Thread [Next in Thread]