diff options
| author | Ben Boeckel <ben.boeckel@kitware.com> | 2018-10-29 19:49:44 (GMT) |
|---|---|---|
| committer | Ben Boeckel <ben.boeckel@kitware.com> | 2018-11-02 14:25:56 (GMT) |
| commit | 85aceda0262bff58a0126b2d4e75756e0507fe3c (patch) | |
| tree | fe919665f22ef165e10e87220b313de593cb0f21 /Source/cmNinjaNormalTargetGenerator.cxx | |
| parent | d697f4eb489ee42a08f2a15efffe55e1c29860b7 (diff) | |
| download | CMake-85aceda0262bff58a0126b2d4e75756e0507fe3c.zip CMake-85aceda0262bff58a0126b2d4e75756e0507fe3c.tar.gz CMake-85aceda0262bff58a0126b2d4e75756e0507fe3c.tar.bz2 | |
cmNinjaNormalTargetGenerator: don't use `|| true` for link-what-you-use
With the `|| true`, a linker error before running link-what-you-use
would also use the `|| true` fragment and unconditionally succeed. Just
skip the addition since `--lwyu=` ignores the return value anyways.
Fixes #18524
Diffstat (limited to 'Source/cmNinjaNormalTargetGenerator.cxx')
| -rw-r--r-- | Source/cmNinjaNormalTargetGenerator.cxx | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/Source/cmNinjaNormalTargetGenerator.cxx b/Source/cmNinjaNormalTargetGenerator.cxx index 6436969..1386706 100644 --- a/Source/cmNinjaNormalTargetGenerator.cxx +++ b/Source/cmNinjaNormalTargetGenerator.cxx @@ -506,7 +506,6 @@ std::vector<std::string> cmNinjaNormalTargetGenerator::ComputeLinkCmd() gt.GetFullPath(cfgName, cmStateEnums::RuntimeBinaryArtifact, /*realname=*/true)); cmakeCommand += targetOutputReal; - cmakeCommand += " || true"; linkCmds.push_back(std::move(cmakeCommand)); } return linkCmds; |