Skip to content

llama-swap: init at 149 #431871

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

06kellyjac
Copy link
Member

@06kellyjac 06kellyjac commented Aug 7, 2025

Things done

Packages llama-swap for nixpkgs

UI also working
image

Not actually put it infront of llama.cpp just yet

Future work: create a module

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Aug 7, 2025
@nixpkgs-ci nixpkgs-ci bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Aug 8, 2025
@xNefas
Copy link

xNefas commented Aug 10, 2025

Having this issue when trying to grab the package from an input of which the URL is the PR's original repository.

nix log /nix/store/8bmwhg1hhbv1if6yh1ifwh6l6rhlncnp-llama-swap-149-go-modules.drv
Running phase: unpackPhase
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking source archive /nix/store/jg46sjjnb3g57hdd3ijv70s5sfb35qgy-source
source root is source
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "patchPhase" }
Executing npmConfigHook
Configuring npm

ERROR: no dependencies were specified
Hint: set `npmDeps` if using these hooks individually. If this is happening with `buildNpmPackage`, please open an issue.

npmDeps is clearly set in the original derivation's definition, yet it fails to build on my system. Refer to my configuration as to how I'm importing it.
rg with 'llama-swap' should find any relevant bits, if you'd like to help me out.

@06kellyjac
Copy link
Member Author

Ah. llama-swap-149-go-modules.drv. This is probably because I had a cached copy of the go modules beforehand.

@06kellyjac
Copy link
Member Author

Force pushed. If you pull it again it should work this time. Thanks for testing @xNefas :)

@anund
Copy link
Contributor

anund commented Aug 12, 2025

Looks like #417516 was working on this but stalled at version 137. (connecting the issues)

@06kellyjac
Copy link
Member Author

Ah, thanks for the link.

@anund
Copy link
Contributor

anund commented Aug 15, 2025

Copied locally and swapped to version 150, works for me. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants