libwebrtc に含まれるバージョン文字列を Android と揃える
#193
Conversation
Sora/PackageInfo.swift
Outdated
| @@ -20,6 +20,9 @@ public enum WebRTCInfo { | |||
| /// WebRTC フレームワークのソースコードのリビジョン | |||
| public static let revision = "6b419a0536b1a0ccfff3682f997c6f19bcbd9bd8" | |||
|
|
|||
| /// WebRTC の branch-heads | |||
| public static let branchHeads = "6261" | |||
There was a problem hiding this comment.
確認するのが遅くなりすいません
この変数名だけ複数系になっているのが気になりました
There was a problem hiding this comment.
libwebrtc の branch-heads から取っています。
https://chromium.googlesource.com/external/webrtc/+/branch-heads/6312
There was a problem hiding this comment.
URI のデザインとして複数形 (branch-heads) になっているのではないかと思っており、 6261 は複数あるものでは無いので少し違和感があります
(https://example.com/posts/1 という URL の 1 を posts と呼んでいるイメージです)
変数名を branchName もしくは branch に変更してはどうかと思いましたが、いかがでしょうか?
There was a problem hiding this comment.
丁寧な説明ありがとうございます。Name らしい値ではないので branch とするようにしました。 8b4949e で対応しています。
Sora/PeerChannel.swift
Outdated
| @@ -307,7 +307,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate { | |||
|
|
|||
| let soraClient = "Sora iOS SDK \(SDKInfo.version)" | |||
|
|
|||
| let webRTCVersion = "Shiguredo-build \(WebRTCInfo.version) (\(WebRTCInfo.version).\(WebRTCInfo.commitPosition).\(WebRTCInfo.maintenanceVersion) \(WebRTCInfo.shortRevision))" | |||
| let webRTCVersion = "Shiguredo-build \(WebRTCInfo.version) (\(String(WebRTCInfo.version.dropFirst())).\(WebRTCInfo.branchHeads).\(WebRTCInfo.commitPosition).\(WebRTCInfo.maintenanceVersion) \(WebRTCInfo.shortRevision))" | |||
There was a problem hiding this comment.
\(String(WebRTCInfo.version.dropFirst()) の String は省けませんか?
https://online.swiftplayground.run/ で確認したところ、なしでも動きそうな気がしました
import Foundation
let greeting = "Hello World"
print("\(greeting.dropFirst())") // ello World
There was a problem hiding this comment.
ありがとうございます。修正して動作確認したところ、問題なさそうでしたのでそうします。
4ae0fda で対応しました
|
ご確認ありがとうございました。マージします。 |
修正内容
シグナリング connect で送信される libwebrtc の内容を Android にあわせました。
当初は branch-head の値が送信されていないことから始まりましたが、
最終的には Android の同項目と出力内容を合わせる対応としています。
Sora C++ SDK も () 内に M をつけていないのでそちらに合わせることにしました。
修正前
"libwebrtc":"Shiguredo-build M122 (M122.1.0 6b419a0)"
修正後
"libwebrtc":"Shiguredo-build M122 (122.6261.1.0 6b419a0)"
Sora Android SDK の出力内容
"libwebrtc": "Shiguredo-build M121 (121.6167.4.0 0f741da)"
確認した内容
This pull request includes changes to the
CHANGES.md,Sora/PackageInfo.swift, andSora/PeerChannel.swiftfiles. The most important changes are aimed at aligning the version string oflibwebrtcin theconnectsignaling message with Android. This involves addingbranch-heads, removing the first character of thelibwebrtcversion in parentheses, and modifying the transmitted string.Here are the key changes:
Version String Alignment:
CHANGES.md: Updated the change log to reflect the alignment of thelibwebrtcversion string in theconnectsignaling message with Android. This involves addingbranch-heads, removing the first character from thelibwebrtcversion in parentheses, and changing the transmitted string fromShiguredo-build M122 (M122.1.0 6b419a0)toShiguredo-build M122 (122.6261.1.0 6b419a0).Code Changes:
Sora/PackageInfo.swift: Addedbranch-headsto theWebRTCInfoenum.Sora/PeerChannel.swift: Modified thewebRTCVersionstring in thePeerChannelclass to include the newbranch-headsand to remove the first character from thelibwebrtcversion.