[cmake-developers] Xcode generator is adding old attributes on .pbxproj

[Arnau Quintana]

Hi Brad,

Thanks for the quick response! :)

The changes to remove “refType” are quite safe. I see no big problems there.

The same for “buildSettings” as this attribute is empty for the “PBXProject” section.


About “buildStyles”, as I see on the cmake code, it is used only to set “COPY_PHASE_STRIP=NO”. I might be missing some reason, but I am not sure cmake should set “NO” by default. But this might be a topic for a separate discussion.

Anyway, on our project we set this property to “NO" for internal builds (debug) and to “YES" for live versions. We do that in our build scripts, using “set_target_property”, but by analysing this issue I found out that this is not working either, and this setting is not correctly set on Xcode.

I do not know if these are related. Any thoughts?

I am curious about what tests are you running. Could you please point me to the relevant tests?

I’ll keep you posted if I do any progress.

Thanks again!
Arnau



On 05/02/16 16:11, "Brad King" <brad.king at kitware.com> wrote:

>On 02/05/2016 09:23 AM, Arnau Quintana wrote:
>> The problem is that the .pbxproj contains some attributes that seem to be deprecated. Specifically:
>> 
>>   * “refType" for the “PBXFileReference" isa
>>   * “buildSettings” and “buildStyles” for “PBXProject" isa
>> 
>> Xcode is nice enough to open the project and regenerate the .pbxproj.
>> 
>> If I open the project with Xcode
>> 
>>   * “refType” is jus removed from the .pbxproj file. Also, a new project does not contain this attribute.
>>   * “buildSettings” and “buildStyles” are left there. But a new project is not creating them.
>> 
>> Are there any ongoing efforts to fix this?
>
>I'm not aware of such effort, and this is the first I've seen a report
>of problems like this.  Over the years I've run the entire test suite
>of CMake with every version of Xcode through 7.2 and have not seen any
>failure due to such content.  I don't normally develop in Xcode though.
>
>> maybe we could remove this, as Xcode 1.5 seems to be old enough
>
>Yes.
>
>> If someone else is working on this, I’d gladly join efforts :) 
>
>If you can provide a patch to fix this, then great!  I've added
>Gregor Jasny to Cc because he may be interested in this too.
>
>Thanks,
>-Brad
>