summaryrefslogtreecommitdiff
path: root/Source
diff options
context:
space:
mode:
authorCristian Adam <cristian.adam@gmail.com>2019-11-22 15:46:27 +0100
committerCristian Adam <cristian.adam@gmail.com>2019-11-25 13:33:44 +0100
commit43ffd2c35caf1df37e915460c05da04b583577c4 (patch)
tree64435ba29f0e9ca440576f5912970cdd79a341b9 /Source
parent602f2118b083f998cd7523efcdd5f39689eb15ac (diff)
downloadcmake-43ffd2c35caf1df37e915460c05da04b583577c4.tar.gz
Unity: No repeated path for internal generated unity files
Fixes: #20002
Diffstat (limited to 'Source')
-rw-r--r--Source/cmLocalGenerator.cxx29
1 files changed, 16 insertions, 13 deletions
diff --git a/Source/cmLocalGenerator.cxx b/Source/cmLocalGenerator.cxx
index 923d2a5df3..2e499b348e 100644
--- a/Source/cmLocalGenerator.cxx
+++ b/Source/cmLocalGenerator.cxx
@@ -3148,6 +3148,22 @@ std::string cmLocalGenerator::GetObjectFileNameWithoutTarget(
}
}
+ // Ensure that for the CMakeFiles/<target>.dir/generated_source_file
+ // we don't end up having:
+ // CMakeFiles/<target>.dir/CMakeFiles/<target>.dir/generated_source_file.obj
+ const char* unitySourceFile = source.GetProperty("UNITY_SOURCE_FILE");
+ const char* pchExtension = source.GetProperty("PCH_EXTENSION");
+ if (unitySourceFile || pchExtension) {
+ if (pchExtension) {
+ customOutputExtension = pchExtension;
+ }
+
+ cmsys::RegularExpression var("(CMakeFiles/[^/]+.dir/)");
+ if (var.find(objectName)) {
+ objectName.erase(var.start(), var.end() - var.start());
+ }
+ }
+
// Replace the original source file extension with the object file
// extension.
bool keptSourceExtension = true;
@@ -3164,19 +3180,6 @@ std::string cmLocalGenerator::GetObjectFileNameWithoutTarget(
}
}
- const char* pchExtension = source.GetProperty("PCH_EXTENSION");
- if (pchExtension) {
- customOutputExtension = pchExtension;
-
- // Make sure that for the CMakeFiles/<target>.dir/cmake_pch.h|xx.c|xx
- // source file, we don't end up having
- // CMakeFiles/<target>.dir/CMakeFiles/<target>.dir/cmake_pch.h|xx.pch
- cmsys::RegularExpression var("(CMakeFiles/[^/]+.dir/)");
- while (var.find(objectName)) {
- objectName.erase(var.start(), var.end() - var.start());
- }
- }
-
// Remove the source extension if it is to be replaced.
if (replaceExt || customOutputExtension) {
keptSourceExtension = false;