Skip to content

deps.ffmpeg: Remove extra parenthesis#311

Open
RytoEX wants to merge 1 commit intoobsproject:masterfrom
RytoEX:fix-extra-parenthesis
Open

deps.ffmpeg: Remove extra parenthesis#311
RytoEX wants to merge 1 commit intoobsproject:masterfrom
RytoEX:fix-extra-parenthesis

Conversation

@RytoEX
Copy link
Copy Markdown
Member

@RytoEX RytoEX commented Apr 7, 2026

Description

Remove extra parenthesis.

Motivation and Context

Noticed this while testing updates to dependencies. The current $Options array results in this set of arguments:

-DUSE_SHARED_MBEDTLS_LIBRARY:BOOL=OFF -DUSE_STATIC_MBEDTLS_LIBRARY:BOOL=ON) -DENABLE_PROGRAMS:BOOL=OFF -DENABLE_TESTING:BOOL=OFF -DGEN_FILES:BOOL=OFF

I thought the parenthesis seemed out of place, then checked, and found there was one extra here.

How Has This Been Tested?

Ran this locally:

.\Build-Dependencies.ps1 -PackageName ffmpeg -Target x64 -Dependencies mbedtls -Configuration Release -Debug

Build succeeded. I did not build OBS Studio against this yet.

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 a review from PatTheMav April 7, 2026 18:33
@RytoEX RytoEX self-assigned this Apr 7, 2026
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.

1 participant