protobuf: remove probe-based library lookup

It is not reliable these days and it is hard to maintain it.
Only leave provider-based lookup or built-in runtime (via
private _linkLibraries property).

Task-number: QBS-1663
Change-Id: I1cc3e3a7b18f721fe252d9b5489ed7495a9a6366
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>

1 files changed, 4 insertions, 96 deletions

diff --git a/share/qbs/modules/protobuf/cpp/protobufcpp.qbs b/share/qbs/modules/protobuf/cpp/protobufcpp.qbs
index e9505dfc4..9a8c55524 100644
--- a/share/qbs/modules/protobuf/cpp/protobufcpp.qbs
+++ b/share/qbs/modules/protobuf/cpp/protobufcpp.qbs
@@ -6,29 +6,14 @@ import "../protobufbase.qbs" as ProtobufBase
 import "../protobuf.js" as HelperFunctions
 
 ProtobufBase {
-    property string includePath: includeProbe.found ? includeProbe.path : undefined
-    property string libraryPath: libraryProbe.found ? libraryProbe.path : undefined
-
     property bool useGrpc: false
 
     property bool _linkLibraries: true
-    property stringList _extraGrpcLibs: []
     readonly property bool _hasModules: protobuflib.present && (!useGrpc || grpcpp.present)
 
-    property string grpcIncludePath: grpcIncludeProbe.found ? grpcIncludeProbe.path : undefined
-    property string grpcLibraryPath: grpcLibraryProbe.found ? grpcLibraryProbe.path : undefined
-
     property string _cxxLanguageVersion: "c++17"
 
-    readonly property string _libraryName: {
-        var libraryName;
-        if (libraryProbe.found) {
-            libraryName = FileInfo.baseName(libraryProbe.fileName);
-            if (libraryName.startsWith("lib"))
-                libraryName = libraryName.substring(3);
-        }
-        return libraryName;
-    }
+    cpp.includePaths: outputDir
 
     Depends { name: "cpp" }
     Depends {
@@ -52,43 +37,6 @@ ProtobufBase {
         names: "grpc_cpp_plugin"
     }
 
-    cpp.libraryPaths: {
-        if (!_linkLibraries || _hasModules)
-            return [];
-
-        var result = [];
-        if (libraryProbe.found)
-            result.push(libraryProbe.path);
-        if (useGrpc && grpcLibraryProbe.found)
-            result.push(grpcLibraryPath);
-        return result;
-    }
-    cpp.dynamicLibraries: {
-        if (!_linkLibraries || _hasModules)
-            return [];
-
-        var result = [];
-        if (_libraryName)
-            result.push(_libraryName)
-        if (qbs.targetOS.includes("unix"))
-            result.push("pthread");
-        if (useGrpc) {
-            result = result.concat(_extraGrpcLibs);
-            result.push("grpc++");
-        }
-        return result;
-    }
-    cpp.includePaths: {
-        if (!_linkLibraries || _hasModules)
-            return [outputDir];
-
-        var result = [outputDir];
-        if (includeProbe.found)
-            result.push(includePath);
-        if (useGrpc && grpcIncludeProbe.found)
-            result.push(grpcIncludePath);
-        return result;
-    }
     cpp.cxxLanguageVersion: _cxxLanguageVersion
 
     Rule {
@@ -123,56 +71,16 @@ ProtobufBase {
         }
     }
 
-    Probes.IncludeProbe {
-        id: includeProbe
-        names: "google/protobuf/message.h"
-        platformSearchPaths: includePath ? [] : base
-        searchPaths: includePath ? [includePath] : []
-    }
-
-    Probes.LibraryProbe {
-        id: libraryProbe
-        names: [
-            "protobuf",
-            "protobufd",
-        ]
-        platformSearchPaths: libraryPath ? [] : base
-        searchPaths: libraryPath ? [libraryPath] : []
-    }
-
-    Probes.IncludeProbe {
-        id: grpcIncludeProbe
-        pathSuffixes: "grpc++"
-        names: "grpc++.h"
-        platformSearchPaths: grpcIncludePath ? [] : base
-        searchPaths: grpcIncludePath ? [grpcIncludePath] : []
-    }
-
-    Probes.LibraryProbe {
-        id: grpcLibraryProbe
-        names: "grpc++"
-        platformSearchPaths: grpcLibraryPath ? [] : base
-        searchPaths: grpcLibraryPath ? [grpcLibraryPath] : []
-    }
-
     validate: {
         HelperFunctions.validateCompiler(compilerName, compilerPath);
 
-        if (_hasModules)
-            return;
-
-        if (_linkLibraries && !includeProbe.found)
-            throw "Can't find cpp protobuf include files. Please set the includePath property.";
-        if (_linkLibraries && !libraryProbe.found)
-            throw "Can't find cpp protobuf library. Please set the libraryPath property.";
+        if (_linkLibraries && ! _hasModules) {
+            throw "Can't find cpp protobuf runtime. Make sure .pc files are present";
+        }
 
         if (useGrpc) {
             if (!File.exists(grpcPluginPath))
                 throw "Can't find grpc_cpp_plugin plugin. Please set the grpcPluginPath property.";
-            if (_linkLibraries && !grpcIncludeProbe.found)
-                throw "Can't find grpc++ include files. Please set the grpcIncludePath property.";
-            if (_linkLibraries && !grpcLibraryProbe.found)
-                throw "Can't find grpc++ library. Please set the grpcLibraryPath property.";
         }
     }
 }