Skip to content

[1/10] Update vendored SPUD to current upstream - #364

Merged
netniV merged 1 commit into
devfrom
refactor/01-spud
Nov 14, 2026
Merged

netniV merged 1 commit into
devfrom
refactor/01-spud

Conversation

@friendly-ripper

Copy link
Copy Markdown
Collaborator

Part 1 of 10 · base dev · 1 commit

The vendored SPUD snapshot is behind upstream, and its install call reports failure poorly. This PR replaces the snapshot with current upstream detour, relocation, and signature-scanning code so later hook work builds on the newer API.

What changes

  • The vendored copy under xmake-packages/packages/s/spud/spud-src/ is replaced. Upstream CMake manifests and the license file stay unchanged, so provenance can still be checked against upstream.
  • Callers and the x64 fixture move to the new install API. Install now reports failures instead of returning a bare pointer, which gives hook code a clear place to stop.
  • The chat title hooks go back to SetWidgetData and OnDidBindContext, on Windows too. The new SPUD patches fit the shared title wrapper, so neither the SetIdentifiers workaround nor the Windows skip is needed.

Testing

Built with xmake. The commit was built on macOS arm64. Nothing was run in the game.

Replace the vendored snapshot with the current upstream detour,
relocation and signature-scanning code. Callers and the x64
fixture move to the new install API, which reports failures
instead of returning a bare pointer.

The upstream CMake manifests and license stay unchanged, so
dependency provenance can still be checked directly.

The chat title hooks go back to SetWidgetData and OnDidBindContext,
on Windows too. The new SPUD patches fit the shared title wrapper,
so neither the SetIdentifiers workaround nor the Windows skip is
needed.
@netniV
netniV added this pull request to the merge queue Nov 14, 2026
Merged via the queue into dev with commit 3e9a408 Nov 14, 2026
6 checks passed
@netniV
netniV deleted the refactor/01-spud branch November 14, 2026 00:06
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