Skip to content

source argument renamed to src for grammar, causes nix flake check failure - #7

Open
kittywitch wants to merge 1 commit into
nvim-neorg:mainfrom
kittywitch:main
Open

source argument renamed to src for grammar, causes nix flake check failure#7
kittywitch wants to merge 1 commit into
nvim-neorg:mainfrom
kittywitch:main

Conversation

@kittywitch

@kittywitch kittywitch commented Jul 13, 2025

Copy link
Copy Markdown

       … from call site
         at /nix/store/h2bn031b0fj0ymv9v7bv5rcdjx58y2l9-source/lib/trivial.nix:989:5:
          988|     # TODO: Should we add call-time "type" checking like built in?
          989|     __functor = self: f;
             |     ^
          990|     __functionArgs = args;

       error: function 'anonymous lambda' called without required argument 'src'
       at /nix/store/h2bn031b0fj0ymv9v7bv5rcdjx58y2l9-source/pkgs/development/tools/parsing/tree-sitter/grammar.nix:10:1:
            9|
           10| {
             | ^
           11|   # language name

This occurs for both tree-sitter-norg-meta and tree-sitter-norg/dev. A similar PR will be made there. I hope this otherwise has no adverse effects for users, but as it stands, the use of https://github.com/nvim-neorg/nixpkgs-neorg-overlay/ currently causes nix flake check to fail for me.

Corresponding other-side PR: nvim-neorg/tree-sitter-norg#77

Cause: NixOS/nixpkgs@5339c7c

Rename of source to src in upstream

@kittywitch kittywitch changed the title Erroneous "source" argument passed instead of "src", causes nix flake check failure source argument renamed to src for grammar, causes nix flake check failure Jul 13, 2025
@kittywitch

Copy link
Copy Markdown
Author

It would probably be better to provide some sort of backwards compatible solution, since this problem and the corresponding issue only arise when you take over the nixpkgs input for this flake. I will look into trying to make one at some point.

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.

1 participant