[131110] trunk/dports/www/serf1/files/patch-SConstruct.diff
jeremyhu at macports.org
jeremyhu at macports.org
Mon Jan 5 23:33:02 PST 2015
Revision: 131110
https://trac.macports.org/changeset/131110
Author: jeremyhu at macports.org
Date: 2015-01-05 23:33:02 -0800 (Mon, 05 Jan 2015)
Log Message:
-----------
serf1: Don't use install_name_tool hack. Set the dylib id correctly at build time
Modified Paths:
--------------
trunk/dports/www/serf1/files/patch-SConstruct.diff
Modified: trunk/dports/www/serf1/files/patch-SConstruct.diff
===================================================================
--- trunk/dports/www/serf1/files/patch-SConstruct.diff 2015-01-06 07:29:35 UTC (rev 131109)
+++ trunk/dports/www/serf1/files/patch-SConstruct.diff 2015-01-06 07:33:02 UTC (rev 131110)
@@ -12,3 +12,36 @@
LIBNAME = 'libserf-%d' % (MAJOR,)
if sys.platform != 'win32':
+--- SConstruct.orig 2015-01-05 22:48:24.000000000 -0800
++++ SConstruct 2015-01-05 22:49:39.000000000 -0800
+@@ -225,7 +225,7 @@ env.Append(RPATH=libdir,
+
+ if sys.platform == 'darwin':
+ # linkflags.append('-Wl,-install_name, at executable_path/%s.dylib' % (LIBNAME,))
+- env.Append(LINKFLAGS='-Wl,-install_name,%s/%s.dylib' % (thisdir, LIBNAME,))
++ env.Append(LINKFLAGS='-Wl,-install_name,%s/%s.dylib' % (libdir, LIBNAME,))
+
+ if sys.platform != 'win32':
+ ### gcc only. figure out appropriate test / better way to check these
+@@ -402,21 +402,6 @@ if CALLOUT_OKAY:
+ install_static = env.Install(libdir, lib_static)
+ install_shared = env.InstallVersionedLib(libdir, lib_shared)
+
+-if sys.platform == 'darwin':
+- # Change the shared library install name (id) to its final name and location.
+- # Notes:
+- # If --install-sandbox=<path> is specified, install_shared_path will point
+- # to a path in the sandbox. We can't use that path because the sandbox is
+- # only a temporary location. The id should be the final target path.
+- # Also, we shouldn't use the complete version number for id, as that'll
+- # make applications depend on the exact major.minor.patch version of serf.
+-
+- install_shared_path = install_shared[0].abspath
+- target_install_shared_path = os.path.join(libdir, '%s.dylib' % LIBNAME)
+- env.AddPostAction(install_shared, ('install_name_tool -id %s %s'
+- % (target_install_shared_path,
+- install_shared_path)))
+-
+ env.Alias('install-lib', [install_static, install_shared,
+ ])
+ env.Alias('install-inc', env.Install(incdir, HEADER_FILES))
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20150105/e8086fde/attachment.html>
More information about the macports-changes
mailing list