Merge topic 'xcode-file-type'
a339ea65
Xcode: Add source file property to control file type (#14854)ae80cb9f
Xcode: Refactor internal source file type selection
This commit is contained in:
commit
567ca4c79e
|
@ -289,6 +289,8 @@ Properties on Source Files
|
|||
/prop_sf/OBJECT_OUTPUTS
|
||||
/prop_sf/SYMBOLIC
|
||||
/prop_sf/WRAP_EXCLUDE
|
||||
/prop_sf/XCODE_EXPLICIT_FILE_TYPE
|
||||
/prop_sf/XCODE_LAST_KNOWN_FILE_TYPE
|
||||
|
||||
Properties on Cache Entries
|
||||
===========================
|
||||
|
|
|
@ -0,0 +1,8 @@
|
|||
XCODE_EXPLICIT_FILE_TYPE
|
||||
------------------------
|
||||
|
||||
Set the Xcode ``explicitFileType`` attribute on its reference to a
|
||||
source file. CMake computes a default based on file extension but
|
||||
can be told explicitly with this property.
|
||||
|
||||
See also :prop_sf:`XCODE_LAST_KNOWN_FILE_TYPE`.
|
|
@ -0,0 +1,9 @@
|
|||
XCODE_LAST_KNOWN_FILE_TYPE
|
||||
--------------------------
|
||||
|
||||
Set the Xcode ``lastKnownFileType`` attribute on its reference to a
|
||||
source file. CMake computes a default based on file extension but
|
||||
can be told explicitly with this property.
|
||||
|
||||
See also :prop_sf:`XCODE_EXPLICIT_FILE_TYPE`, which is preferred
|
||||
over this property if set.
|
|
@ -0,0 +1,7 @@
|
|||
xcode-file-type
|
||||
---------------
|
||||
|
||||
* The :generator:`Xcode` generator learned to check source
|
||||
file properties :prop_sf:`XCODE_EXPLICIT_FILE_TYPE` and
|
||||
:prop_sf:`XCODE_LAST_KNOWN_FILE_TYPE` for a custom Xcode
|
||||
file reference type.
|
|
@ -647,13 +647,14 @@ cmXCodeObject*
|
|||
cmGlobalXCodeGenerator::CreateXCodeSourceFileFromPath(
|
||||
const std::string &fullpath,
|
||||
cmTarget& cmtarget,
|
||||
const std::string &lang)
|
||||
const std::string &lang,
|
||||
cmSourceFile* sf)
|
||||
{
|
||||
// Using a map and the full path guarantees that we will always get the same
|
||||
// fileRef object for any given full path.
|
||||
//
|
||||
cmXCodeObject* fileRef =
|
||||
this->CreateXCodeFileReferenceFromPath(fullpath, cmtarget, lang);
|
||||
this->CreateXCodeFileReferenceFromPath(fullpath, cmtarget, lang, sf);
|
||||
|
||||
cmXCodeObject* buildFile = this->CreateObject(cmXCodeObject::PBXBuildFile);
|
||||
buildFile->SetComment(fileRef->GetComment());
|
||||
|
@ -696,7 +697,7 @@ cmGlobalXCodeGenerator::CreateXCodeSourceFile(cmLocalGenerator* lg,
|
|||
this->CurrentLocalGenerator->GetSourceFileLanguage(*sf);
|
||||
|
||||
cmXCodeObject* buildFile =
|
||||
this->CreateXCodeSourceFileFromPath(sf->GetFullPath(), cmtarget, lang);
|
||||
this->CreateXCodeSourceFileFromPath(sf->GetFullPath(), cmtarget, lang, sf);
|
||||
cmXCodeObject* fileRef = buildFile->GetObject("fileRef")->GetObject();
|
||||
|
||||
cmXCodeObject* settings =
|
||||
|
@ -828,7 +829,8 @@ cmXCodeObject*
|
|||
cmGlobalXCodeGenerator::CreateXCodeFileReferenceFromPath(
|
||||
const std::string &fullpath,
|
||||
cmTarget& cmtarget,
|
||||
const std::string &lang)
|
||||
const std::string &lang,
|
||||
cmSourceFile* sf)
|
||||
{
|
||||
std::string fname = fullpath;
|
||||
cmXCodeObject* fileRef = this->FileRefs[fname];
|
||||
|
@ -848,36 +850,49 @@ cmGlobalXCodeGenerator::CreateXCodeFileReferenceFromPath(
|
|||
}
|
||||
fileRef->AddAttribute("fileEncoding", this->CreateString("4"));
|
||||
|
||||
// Compute the extension.
|
||||
std::string ext;
|
||||
std::string realExt =
|
||||
cmSystemTools::GetFilenameLastExtension(fullpath);
|
||||
if(!realExt.empty())
|
||||
bool useLastKnownFileType = false;
|
||||
std::string fileType;
|
||||
if(sf)
|
||||
{
|
||||
// Extension without the leading '.'.
|
||||
ext = realExt.substr(1);
|
||||
if(const char* e = sf->GetProperty("XCODE_EXPLICIT_FILE_TYPE"))
|
||||
{
|
||||
fileType = e;
|
||||
}
|
||||
|
||||
else if(const char* l = sf->GetProperty("XCODE_LAST_KNOWN_FILE_TYPE"))
|
||||
{
|
||||
useLastKnownFileType = true;
|
||||
fileType = l;
|
||||
}
|
||||
}
|
||||
if(fileType.empty())
|
||||
{
|
||||
// If fullpath references a directory, then we need to specify
|
||||
// lastKnownFileType as folder in order for Xcode to be able to open the
|
||||
// contents of the folder (Xcode 4.6 does not like explicitFileType=folder).
|
||||
// lastKnownFileType as folder in order for Xcode to be able to
|
||||
// open the contents of the folder.
|
||||
// (Xcode 4.6 does not like explicitFileType=folder).
|
||||
if(cmSystemTools::FileIsDirectory(fullpath.c_str()))
|
||||
{
|
||||
fileRef->AddAttribute("lastKnownFileType",
|
||||
this->CreateString("folder"));
|
||||
fileType = "folder";
|
||||
useLastKnownFileType = true;
|
||||
}
|
||||
else
|
||||
{
|
||||
bool keepLastKnownFileType = false;
|
||||
std::string sourcecode = GetSourcecodeValueFromFileExtension(ext,
|
||||
lang, keepLastKnownFileType);
|
||||
const char* attribute = keepLastKnownFileType ?
|
||||
"lastKnownFileType" :
|
||||
"explicitFileType";
|
||||
fileRef->AddAttribute(attribute,
|
||||
this->CreateString(sourcecode.c_str()));
|
||||
// Compute the extension without leading '.'.
|
||||
std::string ext = cmSystemTools::GetFilenameLastExtension(fullpath);
|
||||
if(!ext.empty())
|
||||
{
|
||||
ext = ext.substr(1);
|
||||
}
|
||||
|
||||
fileType = GetSourcecodeValueFromFileExtension(
|
||||
ext, lang, useLastKnownFileType);
|
||||
}
|
||||
}
|
||||
|
||||
fileRef->AddAttribute(useLastKnownFileType? "lastKnownFileType"
|
||||
: "explicitFileType",
|
||||
this->CreateString(fileType));
|
||||
|
||||
// Store the file path relative to the top of the source tree.
|
||||
std::string path = this->RelativeToSource(fullpath.c_str());
|
||||
std::string name = cmSystemTools::GetFilenameName(path.c_str());
|
||||
|
@ -902,7 +917,7 @@ cmGlobalXCodeGenerator::CreateXCodeFileReference(cmSourceFile* sf,
|
|||
this->CurrentLocalGenerator->GetSourceFileLanguage(*sf);
|
||||
|
||||
return this->CreateXCodeFileReferenceFromPath(
|
||||
sf->GetFullPath(), cmtarget, lang);
|
||||
sf->GetFullPath(), cmtarget, lang, sf);
|
||||
}
|
||||
|
||||
//----------------------------------------------------------------------------
|
||||
|
@ -1052,7 +1067,7 @@ cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen,
|
|||
{
|
||||
std::string obj = *oi;
|
||||
cmXCodeObject* xsf =
|
||||
this->CreateXCodeSourceFileFromPath(obj, cmtarget, "");
|
||||
this->CreateXCodeSourceFileFromPath(obj, cmtarget, "", 0);
|
||||
externalObjFiles.push_back(xsf);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -163,10 +163,12 @@ private:
|
|||
std::vector<cmLocalGenerator*>& generators);
|
||||
cmXCodeObject* CreateXCodeFileReferenceFromPath(const std::string &fullpath,
|
||||
cmTarget& cmtarget,
|
||||
const std::string &lang);
|
||||
const std::string &lang,
|
||||
cmSourceFile* sf);
|
||||
cmXCodeObject* CreateXCodeSourceFileFromPath(const std::string &fullpath,
|
||||
cmTarget& cmtarget,
|
||||
const std::string &lang);
|
||||
const std::string &lang,
|
||||
cmSourceFile* sf);
|
||||
cmXCodeObject* CreateXCodeFileReference(cmSourceFile* sf,
|
||||
cmTarget& cmtarget);
|
||||
cmXCodeObject* CreateXCodeSourceFile(cmLocalGenerator* gen,
|
||||
|
|
|
@ -123,6 +123,10 @@ if("${CMAKE_GENERATOR}" MATCHES "Visual Studio [^6]")
|
|||
add_RunCMake_test(SolutionGlobalSections)
|
||||
endif()
|
||||
|
||||
if(XCODE_VERSION AND NOT "${XCODE_VERSION}" VERSION_LESS 3)
|
||||
add_RunCMake_test(XcodeProject)
|
||||
endif()
|
||||
|
||||
add_RunCMake_test(File_Generate)
|
||||
add_RunCMake_test(ExportWithoutLanguage)
|
||||
add_RunCMake_test(target_link_libraries)
|
||||
|
|
|
@ -0,0 +1,3 @@
|
|||
cmake_minimum_required(VERSION 2.8.4)
|
||||
project(${RunCMake_TEST} NONE)
|
||||
include(${RunCMake_TEST}.cmake)
|
|
@ -0,0 +1,3 @@
|
|||
include(RunCMake)
|
||||
|
||||
run_cmake(XcodeFileType)
|
|
@ -0,0 +1,10 @@
|
|||
set(expect-default "explicitFileType = sourcecode")
|
||||
set(expect-explicit "explicitFileType = \"sourcecode.c.h\"")
|
||||
set(expect-lastKnown "lastKnownFileType = \"sourcecode.c.h\"")
|
||||
foreach(src default explicit lastKnown)
|
||||
file(STRINGS ${RunCMake_TEST_BINARY_DIR}/XcodeFileType.xcodeproj/project.pbxproj actual-${src}
|
||||
REGEX "PBXFileReference.*src-${src}")
|
||||
if(NOT actual-${src} MATCHES "${expect-${src}}")
|
||||
message(SEND_ERROR "src-${src} does not match '${expect-${src}}':\n ${actual-${src}}")
|
||||
endif()
|
||||
endforeach()
|
|
@ -0,0 +1,4 @@
|
|||
enable_language(C)
|
||||
add_executable(main main.c src-default src-explicit src-lastKnown)
|
||||
set_property(SOURCE src-explicit PROPERTY XCODE_EXPLICIT_FILE_TYPE sourcecode.c.h)
|
||||
set_property(SOURCE src-lastKnown PROPERTY XCODE_LAST_KNOWN_FILE_TYPE sourcecode.c.h)
|
Loading…
Reference in New Issue