Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: update Nix vendorHash for sqlite v1.50.0 - #434

#434
Merged
SuperCoolPencil merged 1 commit into
SurgeDM:mainSurgeDM/Surge:mainfrom
leomeowqq:fix/nix-vendorhash-updateleomeowqq/Surge:fix/nix-vendorhash-updateCopy head branch name to clipboard
May 10, 2026
Merged

fix: update Nix vendorHash for sqlite v1.50.0#434
SuperCoolPencil merged 1 commit into
SurgeDM:mainSurgeDM/Surge:mainfrom
leomeowqq:fix/nix-vendorhash-updateleomeowqq/Surge:fix/nix-vendorhash-updateCopy head branch name to clipboard

Conversation

@leomeowqq

@leomeowqq leomeowqq commented May 8, 2026

Copy link
Copy Markdown
Contributor

Sqlite was updated from 1.49.1 to 1.50.0, which broke the vendorHash in package.nix.

Fixed it

Greptile Summary

Updates the Nix vendorHash in package.nix to match the new Go vendor tree after sqlite was bumped from v1.49.1 to v1.50.0. The change also reformats the function-argument list to use trailing commas and removes spaces inside the subPackages list literal, bringing the file in line with modern Nix style.

  • vendorHash is replaced with the correct sha256 for the updated dependency tree; without this fix the Nix build fails hash verification.
  • Formatting-only cleanup: function args gain trailing commas and the blank line after }: is removed — no behavior change.

Confidence Score: 5/5

Safe to merge — the change is a mechanical hash update required by the sqlite dependency bump, with no logic changes.

The only substantive change is replacing the vendorHash to match the updated Go vendor tree for sqlite v1.50.0; the accompanying reformatting is purely cosmetic. There is nothing here that could regress runtime behaviour.

No files require special attention.

Important Files Changed

Filename Overview
package.nix Updates vendorHash for sqlite v1.50.0 and applies minor Nix argument-list formatting changes (trailing commas, no spaces inside list brackets)

Reviews (1): Last reviewed commit: "fix: update Nix vendorHash for sqlite v1..." | Re-trigger Greptile

@SuperCoolPencil
SuperCoolPencil merged commit 4ffd75a into SurgeDM:main May 10, 2026
7 checks passed
@SuperCoolPencil

Copy link
Copy Markdown
Member

Thanks! 🚀

StressTestor pushed a commit to StressTestor/Surge that referenced this pull request Jun 1, 2026
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.