Skip to content

Add opt-in SourcePawn 1 compatibility mode - #44

Open
rumblefrog wants to merge 2 commits into
masterfrom
fix/issue-40-oldspcomp
Open

Add opt-in SourcePawn 1 compatibility mode#44
rumblefrog wants to merge 2 commits into
masterfrom
fix/issue-40-oldspcomp

Conversation

@rumblefrog

Copy link
Copy Markdown
Owner

Summary

  • add a use-old-spcomp input for SourceMod 1.13.7451+ while retaining upstream SourcePawn 2 as the default
  • preserve the SourcePawn 2 executables and isolate compiler/proxy cache variants
  • make the Linux proxy safe for quoted paths and arguments
  • cover SourcePawn 1 and 2 selection on Linux and SourcePawn 1 compatibility on Windows

Validation

  • npm run build
  • tsc --noEmit
  • real SourceMod 1.13.7451 SP1/SP2 fixture compilation
  • end-to-end proxy, no-proxy, quoted-path, and cache-reuse checks
  • Windows archive layout and transformation checks

Fixes #40

@Mikusch

Mikusch commented Sep 9, 2026

Copy link
Copy Markdown

What's the advantage of adding use-old-spcomp rather than asking users to explicitly switch to oldspcomp in their workflows? Users who rely on version: '1.13.x' without pinning a specific version should expect potential breaks. Since a workflow change is necessary regardless, wouldn't it be better to add docs on how to use the legacy compiler instead of adding a setting that silently renames binaries?

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.

SM 1.13 compiler builds past 7442 are SourcePawn 2

2 participants