Skip to content

Update dependencies#3351

Open
hwangsihu wants to merge 8 commits into
masterfrom
deps-update
Open

Update dependencies#3351
hwangsihu wants to merge 8 commits into
masterfrom
deps-update

Conversation

@hwangsihu

@hwangsihu hwangsihu commented Jun 30, 2026

Copy link
Copy Markdown
Contributor
  • Update ACE to 8.0.6
  • Update Catch2 to 3.15.1
  • Update FFmpeg to 8.1.2
  • Update JSON-java to 20260522
  • Update libvpx to fcdfe1c29
  • Update OpenSSL to 3.6.3
  • Update PortAudio to f88b584
  • Update WebRTC to R7827

@hwangsihu
hwangsihu requested review from CoBC and bear101 as code owners June 30, 2026 11:14
@CoBC

CoBC commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Do we need to update all these dependencies? Aside from Prism, which has an open issue related to this dependency, what other bugs affecting TeamTalk are fixed by these updates?

@bear101

bear101 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Prism should be updated in a separate PR if it contains important changes. Also Prism is not part of the toolchain.

@hwangsihu we should only update to official release. There no need to update libvpx and PortAudio to some git that may or may not have been tested.

@bear101

bear101 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Please keep this PR then we can merge it when we need to rebuild the toolchain anyway.

@hwangsihu

Copy link
Copy Markdown
Contributor Author

@bear101, Would you mind submitting the patch currently in PortAudio as a PR to the upstream, or would you prefer to handle it yourself? I would like to get your approval since you wrote this patch. It will definitely be helpful if merged.

@hwangsihu

Copy link
Copy Markdown
Contributor Author

I plan to use tags for libvpx when a new version is released. However, this will not be the case for PortAudio. This is because it is too old, to the point where even some distributions have switched to the Git version. However, I will do the same for this when a new version becomes available.

@bear101

bear101 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

@bear101, Would you mind submitting the patch currently in PortAudio as a PR to the upstream, or would you prefer to handle it yourself? I would like to get your approval since you wrote this patch. It will definitely be helpful if merged.

I've submitted it here: PortAudio/portaudio#1160

Unfortunately it has an issue with msys2

@bear101

bear101 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

@bear101, Would you mind submitting the patch currently in PortAudio as a PR to the upstream, or would you prefer to handle it yourself? I would like to get your approval since you wrote this patch. It will definitely be helpful if merged.

@hwangsihu do you have a setup where you can fix the msys2 build error? PortAudio/portaudio#1160

@hwangsihu

Copy link
Copy Markdown
Contributor Author

I have MSYS2. Would you please give me repository permissions?

@hwangsihu

Copy link
Copy Markdown
Contributor Author

I've made the changes. Does the modification look okay? For now, there were no issues on my local MSYS2.

Also, I think we need to modify the code in a way that doesn't break the API. What would be the best way to do that?

@hwangsihu

Copy link
Copy Markdown
Contributor Author

By the way, I recommend switching from Cygwin to MSYS2. Since Cygwin is primarily for Linux emulation and MSYS2 is primarily for development, that would be appropriate.

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.

3 participants