[PR #38] [MERGED] Add support for tvOS #452

Closed
opened 2025-12-29 15:32:13 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/JohnEstropia/CoreStore/pull/38
Author: @Jeehut
Created: 1/17/2016
Status: Merged
Merged: 1/19/2016
Merged by: @JohnEstropia

Base: masterHead: feature/tvOS_support


📝 Commits (2)

  • 9322371 Use tvOS_support feature branch of GCDKit (Carthage + git submodules)
  • 9081b36 Add tvOS target + Configure target + Add shared scheme for tvOS

📊 Changes

6 files changed (+447 additions, -5 deletions)

View changed files

📝 .gitmodules (+1 -1)
📝 Cartfile (+1 -1)
📝 Cartfile.resolved (+1 -1)
📝 Carthage/Checkouts/GCDKit (+1 -1)
📝 CoreStore.xcodeproj/project.pbxproj (+344 -1)
CoreStore.xcodeproj/xcshareddata/xcschemes/CoreStore tvOS.xcscheme (+99 -0)

📄 Description

I did all steps for this project (and even the dependency GCDKit, see here) to work as a tvOS framework and tried to apply the same structure as with the other targets. It's building fine for me so I think this can be merged after a short review. Shouldn't break anything.

I have experienced an issue with the tvOS test target though. I think it's because on tvOS the document paths are a little different. This should be solved but can also be done after merging this – I suggest merging this first and create an issue afterwards. Maybe this pull request form AERecord helps solving the issue, looks similar to me.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/JohnEstropia/CoreStore/pull/38 **Author:** [@Jeehut](https://github.com/Jeehut) **Created:** 1/17/2016 **Status:** ✅ Merged **Merged:** 1/19/2016 **Merged by:** [@JohnEstropia](https://github.com/JohnEstropia) **Base:** `master` ← **Head:** `feature/tvOS_support` --- ### 📝 Commits (2) - [`9322371`](https://github.com/JohnEstropia/CoreStore/commit/9322371224a37514bfdb33df4054795443560901) Use tvOS_support feature branch of GCDKit (Carthage + git submodules) - [`9081b36`](https://github.com/JohnEstropia/CoreStore/commit/9081b36cca31a9a164b307221f52e7ea9af643d2) Add tvOS target + Configure target + Add shared scheme for tvOS ### 📊 Changes **6 files changed** (+447 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `.gitmodules` (+1 -1) 📝 `Cartfile` (+1 -1) 📝 `Cartfile.resolved` (+1 -1) 📝 `Carthage/Checkouts/GCDKit` (+1 -1) 📝 `CoreStore.xcodeproj/project.pbxproj` (+344 -1) ➕ `CoreStore.xcodeproj/xcshareddata/xcschemes/CoreStore tvOS.xcscheme` (+99 -0) </details> ### 📄 Description I did all steps for this project (and even the dependency GCDKit, see [here](https://github.com/JohnEstropia/GCDKit/pull/8)) to work as a tvOS framework and tried to apply the same structure as with the other targets. It's building fine for me so I think this can be merged after a short review. Shouldn't break anything. I have experienced an issue with the tvOS test target though. I think it's because on tvOS the document paths are a little different. This should be solved but can also be done after merging this – I suggest merging this first and create an issue afterwards. Maybe [this](https://github.com/tadija/AERecord/pull/19) pull request form AERecord helps solving the issue, looks similar to me. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 15:32:13 +01:00
adam closed this issue 2025-12-29 15:32:13 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/CoreStore#452