-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
Build failure: envision-unwrapped #372527
Comments
I was wondering whether or not I should bring that patch out of the module. I'll work on fixing this. |
Never mind, the patch isn't applied by the module? I know that I have a module somewhere that does that and it's cursed and I should really make the patched derivation its own package so it gets built by nixpkgs-review. I'll fix the patch, and see how it was possible for this to get broken in the first place. |
The plot thickens. I can't reproduce the failing derivation, even when enabling the envision NixOS module. It looks like you're on a fairly recent nixpkgs; can you update to the latest nixpkgs unstable just to make sure it wasn't a one-off regression? I checked the blame for the file; while I've submitted PRs against envision recently, none of them have been merged. The last version bump was a month ago, and the patch was added since then, iirc. Nothing should have changed to cause the patch to not apply. |
This could be from nixpkgs-xr. Currently the update workflow has been broken for a few days so I wasn't able to catch this. |
Ohh yeah my bad, i forgot i have the nixpkgs-xr NixOS module enabled which also has an option like that and replaces it |
Yeah, just disabled it and tadaaaa it works again, sorry for the unneeded issue |
Steps To Reproduce
Steps to reproduce the behavior:
programs.envision.enable
totrue
Build log
Build Log
Additional context
Metadata
"x86_64-linux"
Linux 6.6.69, NixOS, 25.05 (Warbler), 25.05.20250104.8f3e1f8
yes
yes
nix-env (Nix) 2.24.11
/nix/store/khbvilmsrv4l69nwd52h27j1mp44a0xi-source
Notify maintainers
@Pandapip1
@Scrumplex
Note for maintainers: Please tag this issue in your PR.
Add a 👍 reaction to issues you find important.
The text was updated successfully, but these errors were encountered: