diff options
| author | Craig Scott <craig.scott@crascit.com> | 2019-02-21 12:00:50 (GMT) |
|---|---|---|
| committer | Craig Scott <craig.scott@crascit.com> | 2019-02-21 20:55:56 (GMT) |
| commit | d3b765d4510b73ee17df0e2bee0f7b9a355c5cba (patch) | |
| tree | 61ad8b7782543b8443edf2f222e023362106cc73 /Source/cmInstallTargetGenerator.cxx | |
| parent | e6897c72e7c59f7a0b82ed19c1bdb40d42f7adaa (diff) | |
| download | CMake-d3b765d4510b73ee17df0e2bee0f7b9a355c5cba.zip CMake-d3b765d4510b73ee17df0e2bee0f7b9a355c5cba.tar.gz CMake-d3b765d4510b73ee17df0e2bee0f7b9a355c5cba.tar.bz2 | |
EXCLUDE_FROM_ALL: Don't warn if installing target excluded from all
The original warning pre-dates support for install components.
There are now legitimate scenarios where an install(TARGETS)
command may list a target that is excluded from all, e.g.
hierarchical projects that will never install the component such a
target belongs to.
Fixes: #18938
Diffstat (limited to 'Source/cmInstallTargetGenerator.cxx')
| -rw-r--r-- | Source/cmInstallTargetGenerator.cxx | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/Source/cmInstallTargetGenerator.cxx b/Source/cmInstallTargetGenerator.cxx index 59701a1..9d3a6bb 100644 --- a/Source/cmInstallTargetGenerator.cxx +++ b/Source/cmInstallTargetGenerator.cxx @@ -41,22 +41,6 @@ cmInstallTargetGenerator::cmInstallTargetGenerator( cmInstallTargetGenerator::~cmInstallTargetGenerator() = default; -void cmInstallTargetGenerator::GenerateScript(std::ostream& os) -{ - // Warn if installing an exclude-from-all target. - if (this->Target->GetPropertyAsBool("EXCLUDE_FROM_ALL")) { - std::ostringstream msg; - msg << "WARNING: Target \"" << this->Target->GetName() - << "\" has EXCLUDE_FROM_ALL set and will not be built by default " - << "but an install rule has been provided for it. CMake does " - << "not define behavior for this case."; - cmSystemTools::Message(msg.str(), "Warning"); - } - - // Perform the main install script generation. - this->cmInstallGenerator::GenerateScript(os); -} - void cmInstallTargetGenerator::GenerateScriptForConfig( std::ostream& os, const std::string& config, Indent indent) { |