ENH: Support exporting/importing of Framework targets.

  - Imported frameworks have the FRAMEWORK property set
  - Added cmTarget::IsFrameworkOnApple method to simplify checks
  - Also remove separate IMPORTED_ENABLE_EXPORTS property and just use ENABLE_EXPORTS since, like FRAMEWORK, it just represents the target type.
  - Document FRAMEWORK keyword in INSTALL command.
  - Updated IMPORTED_LOCATION property documentation for Frameworks

1 files changed, 4 insertions, 7 deletions

diff --git a/Source/cmMakefileLibraryTargetGenerator.cxx b/Source/cmMakefileLibraryTargetGenerator.cxx
index 40bbdf2..9ab4649 100644
--- a/Source/cmMakefileLibraryTargetGenerator.cxx
+++ b/Source/cmMakefileLibraryTargetGenerator.cxx
@@ -111,13 +111,11 @@ void cmMakefileLibraryTargetGenerator::WriteStaticLibraryRules()
 //----------------------------------------------------------------------------
 void cmMakefileLibraryTargetGenerator::WriteSharedLibraryRules(bool relink)
 {
-#ifdef __APPLE__
-  if (this->Target->GetPropertyAsBool("FRAMEWORK"))
+  if(this->Target->IsFrameworkOnApple())
     {
     this->WriteFrameworkRules(relink);
     return;
     }
-#endif
   const char* linkLanguage =
     this->Target->GetLinkerLanguage(this->GlobalGenerator);
   std::string linkRuleVar = "CMAKE_";
@@ -479,14 +477,13 @@ void cmMakefileLibraryTargetGenerator::WriteLibraryRules
       outpathImp += "/";
       }
     }
-#if defined(__APPLE__)
+
   // If we're creating a framework, place the output into a framework directory
-  if (this->Target->GetType() == cmTarget::SHARED_LIBRARY &&
-      this->Target->GetPropertyAsBool("FRAMEWORK"))
+  if(this->Target->IsFrameworkOnApple())
     {
     this->CreateFramework(targetName, outpath);
     }
-#endif
+
   std::string targetFullPath = outpath + targetName;
   std::string targetFullPathPDB = outpath + targetNamePDB;
   std::string targetFullPathSO = outpath + targetNameSO;