summaryrefslogtreecommitdiffstats
path: root/Source/cmExportCommand.cxx
diff options
context:
space:
mode:
authorStephen Kelly <steveire@gmail.com>2013-06-18 14:33:33 (GMT)
committerBrad King <brad.king@kitware.com>2013-10-07 23:57:06 (GMT)
commit904ff9fe592882db5dae88e526db2b380d92f87d (patch)
tree2b03c49057bcfb72c69df6148f3028c48a308e30 /Source/cmExportCommand.cxx
parentd4134352abf69374fdad24d53d67734a135daaa5 (diff)
downloadCMake-904ff9fe592882db5dae88e526db2b380d92f87d.zip
CMake-904ff9fe592882db5dae88e526db2b380d92f87d.tar.gz
CMake-904ff9fe592882db5dae88e526db2b380d92f87d.tar.bz2
export: Add policy CMP0024 to disallow include() of export files
Currently, export() is executed at configure-time. One problem with this is that certain exported properties like the link interface may not be complete at the point the export() is encountered leading to an incorrect or incomplete exported representation. Additionally, the generated IMPORTED_LOCATION property may even be incorrect if commands following the export() have an effect on it. Another problem is that it requires the C++ implementation of cmake to be capable of computing the exported information at configure time. This is a limitation on the cleanup and maintenance of the code. At some point in the future, this limitation will be dropped and more implementation will be moved from cmTarget to cmGeneratorTarget.
Diffstat (limited to 'Source/cmExportCommand.cxx')
-rw-r--r--Source/cmExportCommand.cxx2
1 files changed, 2 insertions, 0 deletions
diff --git a/Source/cmExportCommand.cxx b/Source/cmExportCommand.cxx
index f059ceb..4558fb6 100644
--- a/Source/cmExportCommand.cxx
+++ b/Source/cmExportCommand.cxx
@@ -170,6 +170,8 @@ bool cmExportCommand
ebfg.SetCommand(this);
ebfg.SetExportOld(this->ExportOld.IsEnabled());
+ this->Makefile->AddExportedTargetsFile(fname);
+
// Compute the set of configurations exported.
std::vector<std::string> configurationTypes;
this->Makefile->GetConfigurations(configurationTypes);