-
Notifications
You must be signed in to change notification settings - Fork 8
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
libwebrtc を m132.6834.5.3 にアップデートする #227
Conversation
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.
修正ありがとうございます。一点だけ確認をお願いします。
@@ -18,7 +18,7 @@ public enum WebRTCInfo { | |||
public static let commitPosition = "5" | |||
|
|||
/// WebRTC フレームワークのメンテナンスバージョン | |||
public static let maintenanceVersion = "2" | |||
public static let maintenanceVersion = "3" | |||
|
|||
/// WebRTC フレームワークのソースコードのリビジョン | |||
public static let revision = "afaf497805cbb502da89991c2dcd783201efdd08" |
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.
フレームワークのソースコードのリビジョンはアップデートしなくても大丈夫でしょうか?
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.
WebRTC.xcframework/build_info.json を確認したところ同じ値だったので問題ありません!
{
"branch": "M132",
"commit": "5",
"revision": "afaf497805cbb502da89991c2dcd783201efdd08",
"maint": "3"
}
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.
sora-ios-sdk-specs の build_info.json を確認して同じ値だから更新の必要なしということでしょうか?であれば大丈夫です。
ありがとうございます。
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.
webrtc-build で配布されている、webrtc.ios.tar.gz を解凍すると webrtc/Frameworks/WebRTC.xcframework/build_info.json
が入っていて、Sora/PackageInfo.swift
はそこの値をベースにしています。
なので、build_info.json と一致しているので問題ないという意味になります
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.
https://github.com/shiguredo-webrtc-build/webrtc-build の方を見ているのですね。理解できました!
※ README の更新は不要でした
This pull request includes updates to the WebRTC version used in the project. The primary changes involve updating the WebRTC framework version across various configuration files and documentation.
WebRTC Version Update:
CHANGES.md
: Updated to reflect the new WebRTC versionm132.6834.5.3
.Package.swift
: Updated the WebRTC framework version and checksum to132.6834.5.3
. [1] [2]Podfile
: Updated the WebRTC pod version to132.6834.5.3
.Podfile.dev
: Updated the WebRTC pod version to132.6834.5.3
.Sora.podspec
: Updated the WebRTC dependency version to132.6834.5.3
.Sora/PackageInfo.swift
: Updated the WebRTC maintenance version to3
.