From 6cf55644ef9dda817ed88fe70abf9da235de1a1a Mon Sep 17 00:00:00 2001 From: Stephen Kelly Date: Sun, 7 Oct 2012 13:10:24 +0200 Subject: [PATCH] Generate graphviz files at generate time, not configure time. The feature needs access to all link libraries. In the future that will only be possible to calculate at generate-time. Even when the files were generated at configure time, they were generated after user code in CMakeLists files were generated. No policy is needed to handle manipulation of the files from CMake code, because that was never possible. --- Source/cmake.cxx | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Source/cmake.cxx b/Source/cmake.cxx index 290aff05c..8b8cb0101 100644 --- a/Source/cmake.cxx +++ b/Source/cmake.cxx @@ -2437,11 +2437,6 @@ int cmake::ActualConfigure() { this->CacheManager->SaveCache(this->GetHomeOutputDirectory()); } - if ( !this->GraphVizFile.empty() ) - { - std::cout << "Generate graphviz: " << this->GraphVizFile << std::endl; - this->GenerateGraphViz(this->GraphVizFile.c_str()); - } if(cmSystemTools::GetErrorOccuredFlag()) { return -1; @@ -2604,6 +2599,11 @@ int cmake::Generate() return -1; } this->GlobalGenerator->Generate(); + if ( !this->GraphVizFile.empty() ) + { + std::cout << "Generate graphviz: " << this->GraphVizFile << std::endl; + this->GenerateGraphViz(this->GraphVizFile.c_str()); + } if(this->WarnUnusedCli) { this->RunCheckForUnusedVariables();