build: fixed a typo that prevented --bundled-libraries from working correctly

[samba.git] / buildtools / wafsamba / samba_install.py

diff --git a/buildtools/wafsamba/samba_install.py b/buildtools/wafsamba/samba_install.py
index 71d6b86122f2f75d30434defcca13e935f214cc1..28bd270dd4e700adee68b5aa22bc082b5e052038 100644 (file)
--- a/buildtools/wafsamba/samba_install.py
+++ b/buildtools/wafsamba/samba_install.py
@@ -7,6 +7,8 @@ import Options
 from TaskGen import feature, before, after
 from samba_utils import *
 
+O755 = 493
+
 @feature('install_bin')
 @after('apply_core')
 @before('apply_link')
@@ -41,7 +43,7 @@ def install_binary(self):
     # tell waf to install the right binary
     bld.install_as(os.path.join(install_path, orig_target),
                    os.path.join(self.path.abspath(bld.env), self.target),
-                   chmod=0755)
+                   chmod=O755)
 
 
 
@@ -106,20 +108,17 @@ def install_library(self):
 
 ##############################
 # handle the creation of links for libraries and binaries in the build tree
-# note that we use a relative symlink path to allow the whole tree
-# to me moved/copied elsewhere without breaking the links
-t = Task.simple_task_type('symlink_lib', 'rm -f ${LINK_TARGET} && ln -s ${LINK_SOURCE} ${LINK_TARGET}',
-                          shell=True, color='PINK', ext_in='.bin')
-t.quiet = True
 
 @feature('symlink_lib')
 @after('apply_link')
 def symlink_lib(self):
     '''symlink a shared lib'''
-    if Options.is_install:
+
+    if self.target.endswith('.inst'):
         return
 
-    tsk = self.create_task('symlink_lib', self.link_task.outputs[0])
+    blddir = os.path.dirname(self.bld.srcnode.abspath(self.bld.env))
+    libpath = self.link_task.outputs[0].abspath(self.env)
 
     # calculat the link target and put it in the environment
     soext=""
@@ -131,28 +130,29 @@ def symlink_lib(self):
     if link_target == '':
         link_target = '%s/lib%s.so%s' % (LIB_PATH, self.target, soext)
 
+    link_target = os.path.join(blddir, link_target)
 
-    link_source = os_path_relpath(self.link_task.outputs[0].abspath(self.env),
-                                  os.path.join(self.env.BUILD_DIRECTORY, link_target))
-
-    tsk.env.LINK_TARGET = link_target
-    tsk.env.LINK_SOURCE = link_source[3:]
-    debug('task_gen: LINK for %s is %s -> %s',
-          self.name, tsk.env.LINK_SOURCE, tsk.env.LINK_TARGET)
+    if os.path.lexists(link_target):
+        if os.path.islink(link_target) and os.readlink(link_target) == libpath:
+            return
+        os.unlink(link_target)
+    os.symlink(libpath, link_target)
 
 
-t = Task.simple_task_type('symlink_bin', 'rm -f ${BIN_TARGET} && ln -s ${SRC} ${BIN_TARGET}',
-                          shell=True, color='PINK', ext_in='.bin')
-t.quiet = True
-
 @feature('symlink_bin')
 @after('apply_link')
 def symlink_bin(self):
-    '''symlink a binary'''
-    if Options.is_install:
+    '''symlink a binary into the build directory'''
+
+    if self.target.endswith('.inst'):
         return
-    tsk = self.create_task('symlink_bin', self.link_task.outputs[0])
 
-    tsk.env.BIN_TARGET = self.target
-    debug('task_gen: BIN_TARGET for %s is %s', self.name, tsk.env.BIN_TARGET)
+    blddir = os.path.dirname(self.bld.srcnode.abspath(self.bld.env))
+    binpath = self.link_task.outputs[0].abspath(self.env)
+    bldpath = os.path.join(self.bld.env.BUILD_DIRECTORY, self.link_task.outputs[0].name)
 
+    if os.path.lexists(bldpath):
+        if os.path.islink(bldpath) and os.readlink(bldpath) == binpath:
+            return
+        os.unlink(bldpath)
+    os.symlink(binpath, bldpath)