Skip to content

deps.ffmpeg: Backport libvpx assembly bug fix - #327

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
RytoEX:backport-libvpx-asm-bugfix
Oct 22, 2026
Merged

RytoEX merged 1 commit into
obsproject:masterfrom
RytoEX:backport-libvpx-asm-bugfix

Conversation

@RytoEX

@RytoEX RytoEX commented Oct 22, 2026

Copy link
Copy Markdown
Member

Description

deps.ffmpeg: Backport libvpx assembly bug fix

Motivation and Context

Fix an issue between libvpx's assembly, NASM 3.x, and Visual Studio which causes builds to fail in Visual Studio.

https://chromium-review.googlesource.com/c/webm/libvpx/+/8193356

How Has This Been Tested?

I've only tested building libvpx and FFmpeg locally with this change. I haven't yet tested a full obs-studio build.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have read the contributing document.
  • My code has been run through clang-format.
  • My code follows the project's style guidelines
  • My code is not on the master branch.
  • My code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@RytoEX
RytoEX requested review from PatTheMav and Warchamp7 October 22, 2026 17:03
@RytoEX RytoEX self-assigned this Oct 22, 2026
@RytoEX
RytoEX force-pushed the backport-libvpx-asm-bugfix branch from 5cb2e90 to d4c5f22 Compare October 22, 2026 18:01
Fix an issue between libvpx's assembly, NASM 3.x, and Visual Studio
which causes builds to fail in Visual Studio.
@RytoEX
RytoEX force-pushed the backport-libvpx-asm-bugfix branch from d4c5f22 to 6ea6b9c Compare October 22, 2026 18:09
@RytoEX

RytoEX commented Oct 22, 2026

Copy link
Copy Markdown
Member Author

This will probably conflict with #324.

@RytoEX
RytoEX merged commit 65e666c into obsproject:master Oct 22, 2026
22 checks passed
@RytoEX
RytoEX deleted the backport-libvpx-asm-bugfix branch October 22, 2026 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants