Skip to content
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

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

zztkm
Copy link
Contributor

@zztkm zztkm commented Jan 29, 2025

※ README の更新は不要でした

  • [UPDATE] WebRTC m132.6834.5.3 に上げる

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 version m132.6834.5.3.
  • Package.swift: Updated the WebRTC framework version and checksum to 132.6834.5.3. [1] [2]
  • Podfile: Updated the WebRTC pod version to 132.6834.5.3.
  • Podfile.dev: Updated the WebRTC pod version to 132.6834.5.3.
  • Sora.podspec: Updated the WebRTC dependency version to 132.6834.5.3.
  • Sora/PackageInfo.swift: Updated the WebRTC maintenance version to 3.

@zztkm zztkm self-assigned this Jan 29, 2025
@zztkm zztkm requested a review from torikizi January 29, 2025 10:14
Copy link
Contributor

@torikizi torikizi left a 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"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

フレームワークのソースコードのリビジョンはアップデートしなくても大丈夫でしょうか?

Copy link
Contributor Author

@zztkm zztkm Jan 30, 2025

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"
}

Copy link
Contributor

@torikizi torikizi Jan 30, 2025

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 を確認して同じ値だから更新の必要なしということでしょうか?であれば大丈夫です。

ありがとうございます。

Copy link
Contributor Author

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 と一致しているので問題ないという意味になります

Copy link
Contributor

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 の方を見ているのですね。理解できました!

@zztkm zztkm merged commit 6a6bf12 into develop Jan 30, 2025
4 checks passed
@zztkm zztkm deleted the feature/update-libwebrtc-m132.6834.5.3 branch January 30, 2025 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants