-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add tvOS support (dependencies and headers) #1032
base: master
Are you sure you want to change the base?
Changes from all commits
6286898
05976ac
e58b973
4c1d8f1
4c85133
98078f8
79c6a70
aaa3758
0e67c32
e40a4e6
ae04980
5cfb6a8
784ced8
edef34c
7848265
55780ec
68eada8
b110ade
4f94146
478631b
038fc3b
d3f3195
61cfaa6
e9ac62f
1668d6d
2b58b8b
27c3baa
e76eb59
6aaad68
6fd1a09
a4b4302
58b46e0
cf9fa52
4096311
f053da0
bfc7c0d
b402fae
663edf6
cb04124
9a12bca
fffc888
e7cd6e2
be13531
89a9bb4
2ab2b81
72257bf
215088b
0c5c5e2
382d46e
a13df33
8284317
7556b54
475d076
575c79f
fb0f01e
46ab871
6f9fd26
0058f6e
4a5ba40
515ae9b
16b7f82
4a409ea
d56a184
8c135ba
7f86530
dcdc6e7
04e9ffc
66dbe8d
3f84cae
7007d3c
5a50a72
8e3df7a
4853d86
db7afab
bdb0bdd
e4fe2b9
d707ed7
d83d2e6
997ff55
16ae24b
2537817
23fc040
70a4a99
a1c803e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
# Used for HTTP routing | ||
github "marekcirkos/RoutingHTTPServer" | ||
github "shvul/RoutingHTTPServer" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
github "marekcirkos/RoutingHTTPServer" "v1.0.1" | ||
github "shvul/RoutingHTTPServer" "v1.0.2" | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,7 +31,9 @@ | |
@property(getter=isIdleAnimationWaitEnabled) BOOL idleAnimationWaitEnabled; // @synthesize idleAnimationWaitEnabled=_idleAnimationWaitEnabled; | ||
@property(nonatomic) BOOL doesNotHandleUIInterruptions; // @synthesize doesNotHandleUIInterruptions=_doesNotHandleUIInterruptions; | ||
@property(readonly) BOOL fauxCollectionViewCellsEnabled; | ||
@property(readonly, nonatomic) UIInterfaceOrientation interfaceOrientation; //TODO tvos | ||
#if !TARGET_OS_TV | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think TODO can be removed now There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done |
||
@property(readonly, nonatomic) UIInterfaceOrientation interfaceOrientation; | ||
#endif | ||
@property(readonly, nonatomic) BOOL running; | ||
@property(nonatomic) pid_t processID; // @synthesize processID=_processID; | ||
@property(readonly) XCAccessibilityElement *accessibilityElement; | ||
|
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd"> | ||
<plist version="1.0"> | ||
<dict> | ||
<key>IDEDidComputeMac32BitWarning</key> | ||
<true/> | ||
</dict> | ||
</plist> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
<Scheme | ||
LastUpgradeVersion = "0900" | ||
LastUpgradeVersion = "1000" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I this necessary to update this file? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's not necessary. As I remember, I just update repo to "recommended settings". If I revert its update, it will cause merge conflict in further pr's. So, if it is not a serious problem, I will leave this file as it is. It would be updated in next pr's. |
||
version = "1.3"> | ||
<BuildAction | ||
parallelizeBuildables = "YES" | ||
|
@@ -14,56 +14,53 @@ | |
buildForAnalyzing = "YES"> | ||
<BuildableReference | ||
BuildableIdentifier = "primary" | ||
BlueprintIdentifier = "EEF988291C486603005CA669" | ||
BuildableName = "WebDriverAgentRunner.xctest" | ||
BlueprintName = "WebDriverAgentRunner" | ||
BlueprintIdentifier = "EE9B75D31CF7956C00275851" | ||
BuildableName = "IntegrationApp.app" | ||
BlueprintName = "IntegrationApp" | ||
ReferencedContainer = "container:WebDriverAgent.xcodeproj"> | ||
</BuildableReference> | ||
</BuildActionEntry> | ||
</BuildActionEntries> | ||
</BuildAction> | ||
<TestAction | ||
buildConfiguration = "Debug" | ||
selectedDebuggerIdentifier = "" | ||
selectedLauncherIdentifier = "Xcode.IDEFoundation.Launcher.PosixSpawn" | ||
language = "" | ||
systemAttachmentLifetime = "keepNever" | ||
selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB" | ||
selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB" | ||
shouldUseLaunchSchemeArgsEnv = "YES"> | ||
<Testables> | ||
<TestableReference | ||
skipped = "NO"> | ||
<BuildableReference | ||
BuildableIdentifier = "primary" | ||
BlueprintIdentifier = "EEF988291C486603005CA669" | ||
BuildableName = "WebDriverAgentRunner.xctest" | ||
BlueprintName = "WebDriverAgentRunner" | ||
ReferencedContainer = "container:WebDriverAgent.xcodeproj"> | ||
</BuildableReference> | ||
</TestableReference> | ||
</Testables> | ||
<MacroExpansion> | ||
<BuildableReference | ||
BuildableIdentifier = "primary" | ||
BlueprintIdentifier = "EE9B75D31CF7956C00275851" | ||
BuildableName = "IntegrationApp.app" | ||
BlueprintName = "IntegrationApp" | ||
ReferencedContainer = "container:WebDriverAgent.xcodeproj"> | ||
</BuildableReference> | ||
</MacroExpansion> | ||
<AdditionalOptions> | ||
</AdditionalOptions> | ||
</TestAction> | ||
<LaunchAction | ||
buildConfiguration = "Debug" | ||
selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB" | ||
selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB" | ||
language = "" | ||
launchStyle = "0" | ||
useCustomWorkingDirectory = "NO" | ||
ignoresPersistentStateOnLaunch = "NO" | ||
debugDocumentVersioning = "YES" | ||
debugServiceExtension = "internal" | ||
allowLocationSimulation = "YES"> | ||
<MacroExpansion> | ||
<BuildableProductRunnable | ||
runnableDebuggingMode = "0"> | ||
<BuildableReference | ||
BuildableIdentifier = "primary" | ||
BlueprintIdentifier = "EEF988291C486603005CA669" | ||
BuildableName = "WebDriverAgentRunner.xctest" | ||
BlueprintName = "WebDriverAgentRunner" | ||
BlueprintIdentifier = "EE9B75D31CF7956C00275851" | ||
BuildableName = "IntegrationApp.app" | ||
BlueprintName = "IntegrationApp" | ||
ReferencedContainer = "container:WebDriverAgent.xcodeproj"> | ||
</BuildableReference> | ||
</MacroExpansion> | ||
</BuildableProductRunnable> | ||
<AdditionalOptions> | ||
</AdditionalOptions> | ||
</LaunchAction> | ||
|
@@ -73,15 +70,16 @@ | |
savedToolIdentifier = "" | ||
useCustomWorkingDirectory = "NO" | ||
debugDocumentVersioning = "YES"> | ||
<MacroExpansion> | ||
<BuildableProductRunnable | ||
runnableDebuggingMode = "0"> | ||
<BuildableReference | ||
BuildableIdentifier = "primary" | ||
BlueprintIdentifier = "EEF988291C486603005CA669" | ||
BuildableName = "WebDriverAgentRunner.xctest" | ||
BlueprintName = "WebDriverAgentRunner" | ||
BlueprintIdentifier = "EE9B75D31CF7956C00275851" | ||
BuildableName = "IntegrationApp.app" | ||
BlueprintName = "IntegrationApp" | ||
ReferencedContainer = "container:WebDriverAgent.xcodeproj"> | ||
</BuildableReference> | ||
</MacroExpansion> | ||
</BuildableProductRunnable> | ||
</ProfileAction> | ||
<AnalyzeAction | ||
buildConfiguration = "Debug"> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What exactly has been changed in this module that is has to be updated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previous version doesn't have tvOS target, so Carthage cannot build it for our purposes. Also, CocoaHTTPServer lib was updated. I added small fix for tvOS compiling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say it makes sense to create a PR to update the original rep rather than switching to the custom one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The original repo has not been updated for a long time. @marekcirkos made a fork for his fixes. Do you suggest to make pr to his fork?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, this makes sense, because he is anyway the only person who could approve and merge this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mykola-mokhnach here it is: marekcirkos/RoutingHTTPServer#1
Let's continue when @marekcirkos merge it and create new release