-
-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Swift 6.0 CI for building carton for macOS #483
Conversation
なんでLinuxがおるんや |
あ、ここか。エラーの行番号出してくれよなあ。 |
分岐はこれで良いっぽい |
なんだろ?sudoがだめ? |
いけてそう。 |
あれ?パスを通すの忘れたけど、なんか進んでるな |
1e92c2c
to
1dd7798
Compare
- os: macos-14 | ||
xcode: "15.4" | ||
swift: "6.0" | ||
swift-install: |
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.
Xcodeを切り替えるだけでは済まないパターンの場合はこの swift-install
を設定することとします。
@@ -28,6 +34,13 @@ jobs: | |||
- name: Select Xcode version | |||
run: sudo xcode-select --switch /Applications/Xcode_${{ matrix.xcode }}.app/Contents/Developer | |||
|
|||
- name: Install Swift ${{ matrix.swift }} | |||
if: ${{ matrix.swift-install != null }} | |||
run: bash ci/mac/install-swift.bash |
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.
if
で分岐してからスクリプトに入ります。
|
macのCIにSwift6.0を追加します。
そのままだと動かないので↓のハックをCIでインストールするときに適応します。
swiftlang/swift#73327 (comment)
6.0でビルドされたcartonがインストールするSwiftは5.10のままです。
これは別パッチにして後で出します。 #484 で作業しています。
そのため、この変更では、
cartonコードベースがSwift6.0でビルドできるかどうかを検証するだけで、
自動テストで動作検証する時の挙動は5.10のままです。
PRを提出するときに先頭コメントを書き換えました。
後続コメントは作業中のメモなので無視してください。