diff options
| author | Orkun Tokdemir <ilhanorkuntokdemir@gmail.com> | 2024-01-08 13:36:45 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2024-01-09 15:44:43 (GMT) |
| commit | 7198f0d149a05cae8dec3da601e4dd4fc8526276 (patch) | |
| tree | 672973bdc3951a22f2af27416dd9f3e133ac14a9 /Source | |
| parent | 5162ff64d4f069a1fabf0178869ddaaff9045c88 (diff) | |
| download | CMake-7198f0d149a05cae8dec3da601e4dd4fc8526276.zip CMake-7198f0d149a05cae8dec3da601e4dd4fc8526276.tar.gz CMake-7198f0d149a05cae8dec3da601e4dd4fc8526276.tar.bz2 | |
Makefile: Fix double escaping when DEPFILE is used
In commit cfd8a5ac1f (Makefiles: Add support of DEPFILE for
add_custom_command, 2020-12-04, v3.20.0-rc1~237^2~1) we added a
`ConvertToOutputPath` call on a path given to the `depends` field of
`WriteMakeRule`. The latter already handles escaping for Makefile
syntax.
Fixes: #25554
Diffstat (limited to 'Source')
| -rw-r--r-- | Source/cmMakefileTargetGenerator.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/Source/cmMakefileTargetGenerator.cxx b/Source/cmMakefileTargetGenerator.cxx index 17489bd..90afb1b 100644 --- a/Source/cmMakefileTargetGenerator.cxx +++ b/Source/cmMakefileTargetGenerator.cxx @@ -1722,9 +1722,8 @@ void cmMakefileTargetGenerator::GenerateCustomRuleFile( if (!ccg.GetCC().GetDepfile().empty()) { // Add dependency over timestamp file for dependencies management - auto dependTimestamp = cmSystemTools::ConvertToOutputPath( - this->LocalGenerator->MaybeRelativeToTopBinDir( - cmStrCat(this->TargetBuildDirectoryFull, "/compiler_depend.ts"))); + auto dependTimestamp = this->LocalGenerator->MaybeRelativeToTopBinDir( + cmStrCat(this->TargetBuildDirectoryFull, "/compiler_depend.ts")); depends.emplace_back(std::move(dependTimestamp)); } |