Skip to content

deps.ffmpeg: Enable PDBs for FFmpeg builds on Windows#280

Open
RytoEX wants to merge 1 commit into
obsproject:masterfrom
RytoEX:add-ffmpeg-pdbs
Open

deps.ffmpeg: Enable PDBs for FFmpeg builds on Windows#280
RytoEX wants to merge 1 commit into
obsproject:masterfrom
RytoEX:add-ffmpeg-pdbs

Conversation

@RytoEX
Copy link
Copy Markdown
Member

@RytoEX RytoEX commented Mar 3, 2025

Description

Enable PDBs for FFmpeg builds on Windows if the Configuration is set to Debug or RelWithDebInfo.

Motivation and Context

Having PDBs for FFmpeg can be useful for debugging crashes. This was one of the original goals of moving the FFmpeg builds to be natively compiled.

How Has This Been Tested?

Tested locally about a year ago. Should probably be retested.

Types of changes

  • Tweak (non-breaking change to improve existing functionality)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The 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 a review from PatTheMav March 3, 2025 21:31
@RytoEX RytoEX self-assigned this Mar 3, 2025
Comment thread deps.ffmpeg/99-ffmpeg.ps1 Outdated
@RytoEX RytoEX force-pushed the add-ffmpeg-pdbs branch from dd5abd2 to a70121c Compare April 8, 2026 22:36
@RytoEX
Copy link
Copy Markdown
Member Author

RytoEX commented Apr 9, 2026

Just confirming that the recent push still creates PDBs for the DLLs:
image

Executables are suffixed for some reason:
image

The PDBs are not currently copied/installed to the install dir.

Enable PDBs for FFmpeg builds on Windows for all configurations. If the
configuration is set to Release, or RelWithDebInfo, or MinSizeRel,
adjust the linker options accordingly.
@RytoEX RytoEX force-pushed the add-ffmpeg-pdbs branch from a70121c to 79b5a3d Compare April 9, 2026 19:36
@RytoEX
Copy link
Copy Markdown
Member Author

RytoEX commented Apr 9, 2026

Apparently, I had forgotten to actually push the changes. The PDBs are still not actually part of any final artifacts, as I have to figure out where those are installed/copied.

@RytoEX RytoEX requested a review from PatTheMav May 6, 2026 17:53
@PatTheMav
Copy link
Copy Markdown
Member

PatTheMav commented May 6, 2026

Looking at my own code from the distance of a few years makes me want to refactor the entire build system for obs-deps, but that's a different story.. 😅

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