-
-
Notifications
You must be signed in to change notification settings - Fork 187
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
bug: rebuild fails after home manager update #865
Comments
it seems like gnome-text-editor is what's causing the issue https://github.com/danth/stylix/blob/master/modules/gnome-text-editor/common.nix#L17 |
There are a few other modules using nixpkgs overlays too. |
Yea, we will have to change the functionality of all existing overlays. The problem seems to be that using |
@awwpotato A temporary fix could be for you to disable all Home Manager modules that create such overrides. |
Not just disabling them, I’m afraid. I tried setting explicitly My workaround is to actualy prune the three modules that touch overlays in Stylix in a fork, and point to that fork.
EDIT: ah, but in the Stylix docs there is a sample which I can use, muahaha: |
Even disabling modules doesnt seem to work for me imports = [
inputs.stylix.nixosModules.stylix
];
disabledModules = [
# disabled all these paths in both hm and nixos module, maybe this isnt the right one when inside hm?
"${inputs.stylix}/modules/gnome-text-editor/common.nix"
"${inputs.stylix}/modules/gnome-text-editor/hm.nix"
"${inputs.stylix}/modules/gnome-text-editor/nixos.nix"
"${inputs.stylix}/modules/gnome-text-editor/testbed.nix"
]; ❯ nix eval .#nixosConfigurations.nixos-steamdeck-01.config.home-manager.users.my-user.nixpkgs
{ config = null; overlays = [ «lambda @ /nix/store/88ksghk276aj0ms3ny9q9a993w9cvpfm-source/modules/gnome-text-editor/common.nix:18:12» ]; }
# overlay is still there
❯ nixos-rebuild build --flake .#nixos-steamdeck-01 --show-trace
error:
Failed assertions:
- my-user profile: `nixpkgs` options are disabled when `home-manager.useGlobalPkgs` is enabled. |
Same here, @Doosty. I had to actually remove the aforementioned |
Gnome text editor is the only one that applies the overlay to home manager, all the other ones are only for nixos. The temporary fix is to fork stylix and delete the Gnome text editor hm module. |
Using home-manager from yesterday # flake.nix
inputs.home-manager = {
url = "github:nix-community/home-manager/45c07fcf7d28b5fb3ee189c260dee0a2e4d14317";
inputs.nixpkgs.follows = "nixpkgs";
}; |
I'd also appreciate if someone can test nix-community/home-manager#6466, which could be another potential temporary fix. # flake.nix
inputs.home-manager.url = "github:brckd/home-manager/nixpkgs-disabled/warn-instead-of-error"; |
Huh, that's weird. I thought that overrides on home manager and nixos had the same effect, so I included them in both. They should both write to I completely overlooked that home manager configurations often don't have their own nixpkgs instance. I am of course sorry for the inconvenience my uninformed choice caused.
For simplicity, I made one. Haven't tested it though. # flake.nix
inputs.stylix.url = "github:brckd/stylix/gnome-text-editor/remove-home-module"; |
- Workaround for danth/stylix#865 - Use home-manager's git module to configure signing - Disable zellij's bash integration (was changed to enabled by default in a home-manager update) - Remove "permitted insecure packages" after sonarr update - Disable Jellyseerr for now - Dionysus change to color scheme with more contrast
Adds the `stylix.overlays.enable` option which can be disabled to remove all overlays. This is to handle scenarios where overlays cannot be set, because the configuration doesn't handle its own nixpkgs instance. Fixes danth#865.
Does anyone know why? Judging from the code this shouldn't be the case. |
edit: seems like
|
apply this inside my home manager config fixes it, not sure why it doesn't work for @panchoh |
There are a lot of workarounds suggested in this thread, but i'd like to point out that instead of trying to workaround the root of the issue (which involves somewhat nasty things like forking stylix, pinning home-manager, or cherrypicking modules to disable), you can just... forcibly satisfy the assertion. {lib, ...}: {
nixpkgs.overlays = lib.mkForce null;
} Since the option never did anything anyways (in configs where this issue occurs), this won't have any affect on your system. It seems much cleaner than most other proposed workarounds. Literally less than 20 minutes after i posted this, a |
rn i see overlays being used under Ig this should resolve this conflict and and in future if an overlay is needed for any package specific to home-manager only not nixos dependent then |
You're right, the home manager overrides aren't needed most of the time. However, they are needed when one is using Home Manager without Of course, they could just move that package declaration to NixOS. But then they would lose out on the advantage that each user has its own package instance with different overrides. This would be required if say, one user wants to use the I'm open to discussion about whether this admittedly niche use case is worth the potential conflicts with NixOS. #866 would automatically disable the duplicate overrides most of the times it's necessary and give the user the option to manually do so. |
We should keep the home manager overlay for configs with
I don't see how |
Adds the `stylix.overlays.enable` option which can be disabled to remove all overlays. This is to handle scenarios where overlays cannot be set, because the configuration doesn't handle its own nixpkgs instance. Fixes danth#865.
Adds the `stylix.overlays.enable` option which can be disabled to remove all overlays. This is to handle scenarios where overlays cannot be set, because the configuration doesn't handle its own nixpkgs instance. Fixes danth#865.
Adds the `stylix.overlays.enable` option which can be disabled to remove all overlays. This is to handle scenarios where overlays cannot be set, because the configuration doesn't handle its own nixpkgs instance. Fixes danth#865.
Adds the `stylix.overlays.enable` option which can be disabled to remove all overlays. This is to handle scenarios where overlays cannot be set, because the configuration doesn't handle its own nixpkgs instance. Fixes danth#865.
Adds the `stylix.overlays.enable` option which can be disabled to remove all overlays. This is to handle scenarios where overlays cannot be set, because the configuration doesn't handle its own nixpkgs instance. Fixes danth#865.
- Add iwd support, switched over to that - Add back automatic location detection, using beacondb - Silence nixpkgs.overlays warning with stylix, danth/stylix#865 - Update settings for dracula-tmux, use more nerdfonts - Remove rio settings in theme module, included in upstream stylix - Add back wallpaper to lockscreen and make it show all buttons - Make tmux only launch when not in SSH
this commit in home-manager nix-community/home-manager@eb44c16 breaks rebuilding on systems using the home-manager nixos module.
The text was updated successfully, but these errors were encountered: