DaddyFrosty/yoga
1
0
Fork 0
Code Issues 96 Pull Requests 31 Actions Packages Projects Releases 33 Wiki Activity

Add support for Swift Package Manager #1032

Closed
salling wants to merge 17 commits from swiftpm into main
pull from: swiftpm
merge into: DaddyFrosty:main
Conversation 11 Commits 17 Files Changed 3 +36
1 changed files with 36 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 71e687335c - Show all commits

36
Package.swift Normal file
View File

@@ -0,0 +1,36 @@
// swift-tools-version:5.3
cipolleschi commented 2023-06-15 18:02:58 -07:00 (Migrated from github.com)
Review
Copy Link

we can probably use a more recent version of swift-tools here. At least 5.8, until Xcode 15 become stable.

we can probably use a more recent version of swift-tools here. At least `5.8`, until Xcode 15 become stable.
NickGerleman commented 2023-06-16 13:40:16 -07:00 (Migrated from github.com)
Review
Copy Link

We should maybe consider bumping this in CocoaPods as well then, which is currently set on 5.1

We should maybe consider bumping this in CocoaPods as well then, which is currently set on 5.1
// The swift-tools-version declares the minimum version of Swift required to build this package.
import PackageDescription
let package = Package(
name: "yoga",
platforms: [
.iOS(.v14),
],
products: [
.library(
name: "yoga",
targets: ["yoga"]
),
],
targets: [
.target(
name: "yoga",
dependencies: [],
path: "./yoga",
publicHeadersPath: "./yoga",
cxxSettings: [
.headerSearchPath("../"),
.unsafeFlags([
"-fno-omit-frame-pointer",
"-fexceptions",
"-Wall",
"-Werror",
"-std=c++1y",
"-fPIC",
])
]
)
cipolleschi commented 2023-06-15 18:00:46 -07:00 (Migrated from github.com)
Review
Copy Link

this should probably be .cxx17, as this is the version we are typically using internally.

this should probably be `.cxx17`, as this is the version we are typically using internally.
NickGerleman commented 2023-06-16 13:39:44 -07:00 (Migrated from github.com)
Review
Copy Link

Yoga is targeting C++14 for a little bit longer. There was some unhappiness when I tried to bump from C++ 11 directly to 17.

LLVM has had C++ 17 support for almost six years though, so I do plan for us to do a bump later this year. Same rules for same infra, and unlocking more support.

See https://github.com/facebook/yoga/pull/1203 for discussion

Yoga is targeting C++14 for a little bit longer. There was some unhappiness when I tried to bump from C++ 11 directly to 17. LLVM has had C++ 17 support for almost six years though, so I do plan for us to do a bump later this year. Same rules for same infra, and unlocking more support. See https://github.com/facebook/yoga/pull/1203 for discussion
]
)