Skip to content
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

Add ~dblsaiko/nix-extras #698

Closed
wants to merge 2 commits into from
Closed

Add ~dblsaiko/nix-extras #698

wants to merge 2 commits into from

Conversation

2xsaiko
Copy link

@2xsaiko 2xsaiko commented Oct 25, 2023

This adds https://git.sr.ht/~dblsaiko/nix-extras to the index, it's a collection of packages and NixOS modules that I use in my configurations but which are also ready to be used in others'.

@raboof
Copy link
Member

raboof commented Mar 12, 2025

Seems to fail with:

              … while evaluating the module argument `pkgs' in "/nix/store/2pnmvlid82hrccm9qz4qxg8isjlw847n-source/common/modules/programs/kakoune":
      
             error: attribute 'pkgs' missing
             at /nix/store/8rv7qar60c9p7hzwfa02syvl4yzrgmsm-source/lib/modules.nix:554:28:
                553|         addErrorContext (context name)
                554|           (args.${name} or config._module.args.${name})
                   |                            ^
                555|       ) (functionArgs f);

we've seen this in #824 as well. Feel free to open an issue if you think it's a problem in flake-info.

@2xsaiko
Copy link
Author

2xsaiko commented Mar 12, 2025

Yeah, this seems like a bug in flake-info. Looks like the pkgs argument isn't being passed to the module loader for some reason. That module (kakoune) is fine, I use it myself.

@2xsaiko
Copy link
Author

2xsaiko commented Mar 12, 2025

Looking at it more closely, I believe this is related to usage of pkgs.formats.*, see the issue description.

@raboof
Copy link
Member

raboof commented Apr 2, 2025

let's close this for now until someone has fixed #902 then

@raboof raboof closed this Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants