(0039705)
Gregor Jasny (developer)
2015-10-29 16:40

Diff in Xcode project after upgrade:

@@ -3255,6 +3256,7 @@
         91E22C98754C40059A395810 /* Debug */ = {
             isa = XCBuildConfiguration;
             buildSettings = {
+				ENABLE_TESTABILITY = YES;
                 ONLY_ACTIVE_ARCH = YES;
                 SDKROOT = /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk;
                 
SYMROOT = /Users/jasny/src/wdt/_build/build;

(0039706)
Gregor Jasny (developer)
2015-10-29 17:33
edited on: 2015-10-29 17:34

Xcode expects the "Testability" attribute to be set in the Debug configuration. The best thing would be if the Xcode SDKs would provide these settings by default.

The problem I see here is that 'Debug' is a arbitrary configuration. So I'm a little bit reluctant to set it by default. With 0014947 fixed you would be able to set it in the project itself:

set(CMAKE_XCODE_ATTRIBUTE_ENABLE_TESTABILITY[variant=Debug] "YES")

Another option would be to set LastUpgradeCheck to 710 to prevent the notification.

@@ -1145,6 +1145,7 @@
             isa = PBXProject;
             attributes = {
                 BuildIndependentTargetsInParallel = YES;
+				LastUpgradeCheck = 0710;
             };
             buildConfigurationList = 35CF513637874DB691923438 /* Build configuration list for PBXProject 
"WDT" */;
             compatibilityVersion = "Xcode 3.2";

Brad, what do you think?

PS: This setting seems to make only a difference for Swift

(0039709)
Brad King (manager)
2015-10-30 08:49

Re 0015817:0039706: Since our Xcode project files are generated there is no need for them ever to be upgraded automatically by Xcode. Any updates that are needed should be taught to CMake's generator. Therefore I think we should always generate LastUpgradeCheck to match the version of Xcode for which we are generating. Thanks!

(0039869)
Brad King (manager)
2015-11-13 08:27

Fix is here:

Xcode: Set LastUpgradeCheck to current Xcode version
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=13610474 [^]

(0040960)
Robert Maynard (manager)
2016-05-02 08:30

Closing resolved issues that have not been updated in more than 4 months.

Issue History
Date Modified	Username	Field	Change
2015-10-28 18:59	Laurent Demailly	New Issue
2015-10-29 08:24	Brad King	Assigned To	=> Gregor Jasny
2015-10-29 08:24	Brad King	Status	new => assigned
2015-10-29 16:40	Gregor Jasny	Note Added: 0039705
2015-10-29 17:33	Gregor Jasny	Note Added: 0039706
2015-10-29 17:34	Gregor Jasny	Note Edited: 0039706
2015-10-30 08:49	Brad King	Note Added: 0039709
2015-11-13 08:25	Gregor Jasny	Status	assigned => resolved
2015-11-13 08:25	Gregor Jasny	Fixed in Version	=> CMake 3.5
2015-11-13 08:25	Gregor Jasny	Resolution	open => fixed
2015-11-13 08:27	Brad King	Note Added: 0039869
2016-05-02 08:30	Robert Maynard	Note Added: 0040960
2016-05-02 08:30	Robert Maynard	Status	resolved => closed

Notes
(0039705) Gregor Jasny (developer) 2015-10-29 16:40	Diff in Xcode project after upgrade: @@ -3255,6 +3256,7 @@ 91E22C98754C40059A395810 /* Debug */ = { isa = XCBuildConfiguration; buildSettings = { + ENABLE_TESTABILITY = YES; ONLY_ACTIVE_ARCH = YES; SDKROOT = /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk; SYMROOT = /Users/jasny/src/wdt/_build/build;

(0039706) Gregor Jasny (developer) 2015-10-29 17:33 edited on: 2015-10-29 17:34	Xcode expects the "Testability" attribute to be set in the Debug configuration. The best thing would be if the Xcode SDKs would provide these settings by default. The problem I see here is that 'Debug' is a arbitrary configuration. So I'm a little bit reluctant to set it by default. With 0014947 fixed you would be able to set it in the project itself: set(CMAKE_XCODE_ATTRIBUTE_ENABLE_TESTABILITY[variant=Debug] "YES") Another option would be to set LastUpgradeCheck to 710 to prevent the notification. @@ -1145,6 +1145,7 @@ isa = PBXProject; attributes = { BuildIndependentTargetsInParallel = YES; + LastUpgradeCheck = 0710; }; buildConfigurationList = 35CF513637874DB691923438 /* Build configuration list for PBXProject "WDT" */; compatibilityVersion = "Xcode 3.2"; Brad, what do you think? PS: This setting seems to make only a difference for Swift

(0039709) Brad King (manager) 2015-10-30 08:49	Re 0015817:0039706: Since our Xcode project files are generated there is no need for them ever to be upgraded automatically by Xcode. Any updates that are needed should be taught to CMake's generator. Therefore I think we should always generate LastUpgradeCheck to match the version of Xcode for which we are generating. Thanks!

(0039869) Brad King (manager) 2015-11-13 08:27	Fix is here: Xcode: Set LastUpgradeCheck to current Xcode version https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=13610474 [^]

(0040960) Robert Maynard (manager) 2016-05-02 08:30	Closing resolved issues that have not been updated in more than 4 months.

Relationships