Skip to content

Remove 32-bit server support & enforce W10+#4977

Open
Lpsd wants to merge 8 commits into
multitheftauto:masterfrom
Lpsd:nuke-32-plus-nt6
Open

Remove 32-bit server support & enforce W10+#4977
Lpsd wants to merge 8 commits into
multitheftauto:masterfrom
Lpsd:nuke-32-plus-nt6

Conversation

@Lpsd

@Lpsd Lpsd commented Jun 22, 2026

Copy link
Copy Markdown
Member

Summary

This PR removes 32-bit server support from the build system in preparation for the EHS to cpp-httplib replacement (#4940).

Changes include dropping Linux x86 server platforms, removing Windows 32-bit server project builds, bumping the server minimum Windows version to 10, and cleaning up dead code for pre-Win10 OS compatibility paths in the client.

Motivation

cpp-httplib does not officially support 32-bit platforms.
Rather than maintain unsupported configurations we are dropping 32-bit server builds entirely, see additional reasoning: #4940 (comment)
The client remains 32-bit (obviously) and the installer gate for pre-Win10 is already in place for 1.7 release.
This cleanup ensures no 32-bit server binaries are built going forward.

Test plan

Verified that premake5 generates correct project files for both Windows (x86 x64 arm64) and Linux (x64 arm arm64) without the x86 Linux platform.
Confirmed that Windows x86 builds still compile the client while server projects are excluded via ExcludeFromBuild.
The server launcher now checks for Windows 10 or later at startup and the installer no longer offers the 32-bit server executable or Win7 SHA2 KB patches.

Checklist

  • Your code should follow the coding guidelines.
  • Smaller pull requests are easier to review. If your pull request is beefy, your pull request should be reviewable commit-by-commit.

Comment thread Client/core/CSettings.cpp
vecTemp.fY += fLineHeight;

// Windows 8 compatibility
m_pWin8Label = reinterpret_cast<CGUILabel*>(pManager->CreateLabel(pTabAdvanced, _("Windows 8 compatibility:")));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not entirely sure if these should be removed. Looks like it was added for forward compatiiblity with win8.

Lpsd@e619d6f

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

These options are not visible for users running W10+

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.

2 participants