From 3b99b3a9f02af061f06a21e26f279b2ae2790810 Mon Sep 17 00:00:00 2001
From: dec05eba <dec05eba@protonmail.com>
Date: Sun, 20 Jan 2019 04:18:35 +0100
Subject: Fix rpath for dynamic library dependencies for tests, cleanup empty
 rpath lib

---
 backend/ninja/Ninja.cpp | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

(limited to 'backend/ninja/Ninja.cpp')

diff --git a/backend/ninja/Ninja.cpp b/backend/ninja/Ninja.cpp
index 3d4039f..5d1a506 100644
--- a/backend/ninja/Ninja.cpp
+++ b/backend/ninja/Ninja.cpp
@@ -715,9 +715,12 @@ namespace backend
         string result;
         for(const string &flag : linkerFlags)
         {
-            if(!result.empty())
+            string dynamicLibFullPath = extractDynamicLibDirsFromLinkerFlag(flag);
+            if(!result.empty() && !dynamicLibFullPath.empty())
                 result += ":";
-            result += extractDynamicLibDirsFromLinkerFlag(flag);
+            
+            if(!dynamicLibFullPath.empty())
+                result += dynamicLibFullPath;
         }
         return result;
     }
@@ -1677,14 +1680,14 @@ namespace backend
             }
         }
 
-        Result<bool> buildTestResult = buildTests(allLinkerFlags, projectGeneratedBinary, config, cflags, dependencyExportIncludeDirs);
+        Result<bool> buildTestResult = buildTests(allLinkerFlags, projectGeneratedBinary, config, cflags, dependencyExportIncludeDirs, dynamicLinkerFlags);
         if(!buildTestResult)
             return buildTestResult;
 
         return Result<bool>::Ok(true);
     }
 
-    Result<bool> Ninja::buildTests(const std::string &parentLinkerFlags, const std::string &parentGeneratedLib, const SibsConfig &config, const vector<ninja::NinjaArg> &parentCflags, const string &parentDependencyExportIncludeDirs)
+    Result<bool> Ninja::buildTests(const std::string &parentLinkerFlags, const std::string &parentGeneratedLib, const SibsConfig &config, const vector<ninja::NinjaArg> &parentCflags, const string &parentDependencyExportIncludeDirs, const std::vector<std::string> &parentDynamicLinkerFlags)
     {
         if(testSourceDir.empty() || !config.shouldBuildTests())
             return Result<bool>::Ok(true);
@@ -1731,6 +1734,11 @@ namespace backend
         if(!parentLinkerFlags.empty())
             ninja.addDependency(parentLinkerFlags);
 
+        for(const std::string &parentDynamicLinkerFlag : parentDynamicLinkerFlags)
+        {
+            ninja.addDynamicDependency(parentDynamicLinkerFlag);
+        }
+
         switch(config.getPackageType())
         {
             case PackageType::STATIC:
-- 
cgit v1.2.3