Skip to content

fix(ci): Update for SM 1.13 - #4

Merged
Rushaway merged 1 commit into
srcdslab:masterfrom
Rushaway:fixci113
Jul 31, 2026
Merged

fix(ci): Update for SM 1.13#4
Rushaway merged 1 commit into
srcdslab:masterfrom
Rushaway:fixci113

Conversation

@Rushaway

Copy link
Copy Markdown
Member

This pull request updates the build scripts to support newer versions of SourceMod, Metamod:Source, and C++ compilers, and adds support for the SafetyHook library if present. It also modernizes the build requirements to require C++17 and updates platform-specific build flags for better compatibility and performance.

Build System Modernization and Dependency Updates:

  • Updated SourceMod and Metamod:Source detection to prefer newer versions (sourcemod-1.13 and mmsource-1.12) and improved fallback logic for finding their source directories. [1] [2]
  • Changed the CI workflow to build against the latest sourcemod master branch instead of the old 1.11-dev branch.

Compiler and Platform Support:

  • Increased minimum supported compiler versions to require full C++17 support: MSVC 2017 15.7+ (_MSC_VER 1914+), GCC 9+, and Clang 5+. Updated all relevant C++ flags to use -std=c++17 or /std:c++17. [1] [2] [3]
  • Updated macOS deployment target to 10.15 and added/adjusted platform-specific defines for Linux and macOS. [1] [2]

SafetyHook Integration:

  • Added logic to detect and integrate the SafetyHook library if present in the SourceMod public directory. This includes copying the library locally, building it as a dependency, and linking it in place of legacy detour code. [1] [2] [3] [4]

Build Flags and Warning Handling:

  • Updated and cleaned up compiler warning flags, removed deprecated flags, and improved handling of architecture-specific and compiler-specific options. [1] [2]

These changes ensure the project builds with modern toolchains and dependencies, and can take advantage of the SafetyHook library for detouring if available.

@Rushaway
Rushaway merged commit a949f55 into srcdslab:master Jul 31, 2026
2 checks passed
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