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

Remove non-essential Windows x86 deps builds #196

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

PatTheMav
Copy link
Member

Description

Removes x86 slices of most Windows dependencies, with the exception of Detours and Vulkan, which are both needed for capture plugins.

Motivation and Context

Deprecate all traces of standalone x86 OBS Studio versions.

How Has This Been Tested?

Tested on Windows 11 VM.

Types of changes

  • Code cleanup (non-breaking change which makes code smaller or more readable)

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 changed the title Windows x86 deprecation Remove non-essential Windows x86 deps builds Jun 30, 2023
Copy link
Member

@RytoEX RytoEX left a comment

Choose a reason for hiding this comment

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

Do the first two commits individually work in the absence of the third commit? It looks like they should, but I wanted to confirm.

Commit message nit:

  • detours -> Detours

@PatTheMav
Copy link
Member Author

Do the first two commits individually work in the absence of the third commit? It looks like they should, but I wanted to confirm.

Commit message nit:

  • detours -> Detours

They should, yeah.

Copy link
Member

@RytoEX RytoEX left a comment

Choose a reason for hiding this comment

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

Seems fine. The one thing to watch out for is that it will become impossible to do full 32-bit builds in legacy CMake, so it may be helpful (though perhaps not required) to have instructions for a slim 32-bit build with just the win-capture and win-dshow components.

@RytoEX
Copy link
Member

RytoEX commented Jul 17, 2023

Per off-thread discussion, we're going to hold off on merging this change because removing the x86 Windows deps would make doing complete Windows legacy CMake builds of obs-studio a bit burdensome. We'll remove the x86 Windows deps once we remove the legacy CMake.

@RytoEX RytoEX marked this pull request as draft July 17, 2023 15:29
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.

None yet

2 participants