Browse Source

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
pull/324/head
Ben Boeckel 7 years ago
parent
commit
85aceda026
  1. 1
      Source/cmNinjaNormalTargetGenerator.cxx

1
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;

Loading…
Cancel
Save