Skip to content

fix(nix): support skia-bindings v0.99.0#171

Merged
m3nu merged 2 commits into
borgbase:mainfrom
jonknapp:fix-nix-flake-skia-bindings
Jul 12, 2026
Merged

fix(nix): support skia-bindings v0.99.0#171
m3nu merged 2 commits into
borgbase:mainfrom
jonknapp:fix-nix-flake-skia-bindings

Conversation

@jonknapp

@jonknapp jonknapp commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Fixes #170

@m3nu m3nu merged commit 6ddd104 into borgbase:main Jul 12, 2026
@m3nu

m3nu commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Thanks for the version bump! I'm also adding a CI job to test those in the future.

m3nu added a commit that referenced this pull request Jul 12, 2026
Verifies all pinned flake inputs (narHash check for every input,
including the three skia-binaries tarballs), evaluates flake outputs
for all systems, and builds the three packages on x86_64-linux.
Paths-filtered so it only runs when the flake, nix definitions, or
Cargo manifests change. Closes the gap where lock-file PRs like #171
had no machine verification.
m3nu added a commit that referenced this pull request Jul 12, 2026
* ci: add nix flake verification and build job

Verifies all pinned flake inputs (narHash check for every input,
including the three skia-binaries tarballs), evaluates flake outputs
for all systems, and builds the three packages on x86_64-linux.
Paths-filtered so it only runs when the flake, nix definitions, or
Cargo manifests change. Closes the gap where lock-file PRs like #171
had no machine verification.

* ci(nix): fail fast when skia-binaries pins drift from Cargo.lock

skia-bindings' build script does not validate the archive supplied via
SKIA_BINARIES_URL, so a stale flake pin silently builds against outdated
skia binaries instead of failing the build.
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.

nix flake support breaks when skia-bindings is updated

2 participants