Skip to content

Fix PEPPERED asl-help dependency download - #2929

Merged
wooferzfg merged 1 commit into
LiveSplit:masterfrom
WDN2010:fix/peppered-asl-help-dependency
Sep 27, 2026
Merged

wooferzfg merged 1 commit into
LiveSplit:masterfrom
WDN2010:fix/peppered-asl-help-dependency

Conversation

@WDN2010

@WDN2010 WDN2010 commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to merged #2924: add the missing pinned asl-help dependency URL to PEPPERED's registry entry. The published ASL reads Components/asl-help at startup and rejects anything other than SHA-256 c0ece0762d65cb831082a465af2c2cd64cc745d5ede2b7eb339fb84030cb1fb5, but the current registry entry downloads only the ASL and the project reader DLL. A clean registry installation can therefore lack the required helper.

The new third URL is the same immutable upstream commit already used by Nunholy's registry entry. No ASL, reader DLL, timing logic, hash checks, or other registry entries change.

Verification

  • Parsed the full upstream XML and confirmed exactly one line added; all 2,639 AutoSplitter records remain present and Nunholy's URL list is unchanged.
  • Confirmed URL order: PEPPERED ASL → Peppered.AutoSplitter.dll → pinned asl-help.
  • Fetched the exact upstream helper URL (HTTP 200) and checked its SHA-256 against the published PEPPERED ASL.
  • git diff --check passes. Windows clean registry-download/activation should be checked separately; this change does not solve the global shared Components/asl-help filename collision.

Contributor checklist

  • This change adds only a pinned dependency URL to my existing splitter; it contains no malicious functionality, and I remain responsible for its behavior.
  • It does not replace another author's Auto Splitter.
  • The PEPPERED splitter source is MIT-licensed and may be forked/hosted; asl-help is a separately licensed upstream dependency with corresponding source at the pinned commit.

@wooferzfg
wooferzfg added this pull request to the merge queue Sep 27, 2026
Merged via the queue into LiveSplit:master with commit 0f6865a Sep 27, 2026
1 check passed
@WDN2010
WDN2010 deleted the fix/peppered-asl-help-dependency branch September 27, 2026 14:52
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