Philip Taron
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.
Thank you, Peter @peterhoeg
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.