From 1c5bfab532e96bc728691a15b092d388a9e0a0d5 Mon Sep 17 00:00:00 2001 From: Brad King Date: Tue, 6 Mar 2018 14:00:54 -0500 Subject: cmTarget: Simplify ClearDependencyInformation implementation Remove the `_LIB_DEPENDS` cache entry unconditionally. It will be populated again later if needed. There is no need to ask the user to remove the cache entry if the target type changes. Issue: #16364 --- Source/cmTarget.cxx | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/Source/cmTarget.cxx b/Source/cmTarget.cxx index f7ead9f..b868e84 100644 --- a/Source/cmTarget.cxx +++ b/Source/cmTarget.cxx @@ -637,24 +637,9 @@ const std::vector& cmTarget::GetLinkDirectories() const void cmTarget::ClearDependencyInformation(cmMakefile& mf) { - // Clear the dependencies. The cache variable must exist iff we are - // recording dependency information for this target. std::string depname = this->GetName(); depname += "_LIB_DEPENDS"; - if (this->RecordDependencies) { - mf.AddCacheDefinition(depname, "", "Dependencies for target", - cmStateEnums::STATIC); - } else { - if (mf.GetDefinition(depname)) { - std::string message = "Target "; - message += this->GetName(); - message += " has dependency information when it shouldn't.\n"; - message += "Your cache is probably stale. Please remove the entry\n "; - message += depname; - message += "\nfrom the cache."; - cmSystemTools::Error(message.c_str()); - } - } + mf.RemoveCacheDefinition(depname); } std::string cmTarget::GetDebugGeneratorExpressions( -- cgit v0.12