CMake: Fix working directory for targets in fileapi mode

Prepend the full path to the build directory.

Change-Id: I55111b656fd4b99ee68517c09117f142e88947b1
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
This commit is contained in:
Tobias Hunger
2019-07-26 11:37:13 +02:00
parent d2796546a4
commit 4daba5a9fa

View File

@@ -172,8 +172,8 @@ QList<CMakeBuildTarget> generateBuildTargets(const PreprocessedData &input,
QDir sourceDir(sourceDirectory.toString());
QDir buildDir(buildDirectory.toString());
const QList<CMakeBuildTarget> result = transform<
QList>(input.targetDetails, [&sourceDir, &buildDir](const TargetDetails &t) -> CMakeBuildTarget {
const QList<CMakeBuildTarget> result = transform<QList>(
input.targetDetails, [&sourceDir, &buildDir](const TargetDetails &t) -> CMakeBuildTarget {
CMakeBuildTarget ct;
ct.title = t.name;
ct.executable = t.artifacts.isEmpty()
@@ -192,12 +192,9 @@ QList<CMakeBuildTarget> generateBuildTargets(const PreprocessedData &input,
else
type = UtilityType;
ct.targetType = type;
if (t.artifacts.isEmpty()) {
ct.workingDirectory = t.buildDir;
} else {
ct.workingDirectory = FilePath::fromString(QDir::cleanPath(
QDir(t.buildDir.toString()).absoluteFilePath(t.artifacts.at(0).toString() + "/..")));
}
ct.workingDirectory = ct.executable.isEmpty() ? FilePath::fromString(
buildDir.absoluteFilePath(t.buildDir.toString()))
: ct.executable.parentDir();
ct.sourceDirectory = FilePath::fromString(
QDir::cleanPath(sourceDir.absoluteFilePath(t.sourceDir.toString())));
@@ -205,7 +202,8 @@ QList<CMakeBuildTarget> generateBuildTargets(const PreprocessedData &input,
const BacktraceNode &node = t.backtraceGraph.nodes[static_cast<size_t>(t.backtrace)];
const int fileIndex = node.file;
if (fileIndex >= 0) {
ct.definitionFile = FilePath::fromString(QDir::cleanPath(sourceDir.absoluteFilePath(
ct.definitionFile = FilePath::fromString(
QDir::cleanPath(sourceDir.absoluteFilePath(
t.backtraceGraph.files[static_cast<size_t>(fileIndex)])));
ct.definitionLine = node.line;
}