Philip Taron

Results 28 comments of Philip Taron

Hmmm. I wonder if we could abstract this with a `tag` argument to `fetchFromGitHub` which understands `refs/tags`. That would eliminate both the `refs/tags` in the `src` and also the `refs/tags`...

nixpkgs carries this patch since Oct 20, 2022: - https://github.com/NixOS/nixpkgs/pull/195715

Tested. Nice. Let's do a `nix flake update` too. I have on my list to set up an automated PR system to push nix flake updates since https://github.com/ggml-org/llama.cpp/pull/10526 landed.

I won't be able to work on this, but I will gladly test and merge PRs that do.

Does the new builtins.warn help with this? https://nix.dev/manual/nix/2.23/release-notes/rl-2.23

That's a worthwhile Nix quality of life feature. I'm assuming there's no way for evaluation or lib.warn to understand that it's being called in a `nix search` context?

Sorry, I've been consumed with work that's not nixpkgs recently. Thanks for your patience.