| 29 Jun 2025 |
ElvishJerricco | it's literally the only thing nixos-generate-config is about? | 04:36:02 |
emily | "I want DHCP by default" is configuration.nix not hardware-configuration.nix teritory | 04:36:04 |
hexa | Whatever we give as a configuration.nix is mutable by default | 04:36:14 |
emily | and there's no promise that configuration.nix never has to change | 04:36:15 |
hexa | And everyone manages to enable network manager on their laptop as well | 04:36:29 |
hexa | or not, idk | 04:36:32 |
emily | In reply to @elvishjerricco:matrix.org it's literally the only thing nixos-generate-config is about? it generates a configuration.nix too, does it not? | 04:36:40 |
emily | I haven't actually run it in years | 04:36:46 |
ElvishJerricco | I mean if you just want to move all networking stuff out of hardware-configuration.nix altogether that's a different thing. | 04:36:52 |
hexa | Redacted or Malformed Event | 04:36:59 |
hexa | Redacted or Malformed Event | 04:37:02 |
ElvishJerricco | networking.useDHCP = lib.mkDefault true; goes in hardware-configuration.nix | 04:37:27 |
hexa | sorry, I might be wrong | 04:37:31 |
hexa | that's stupid to have in hardware parts | 04:37:40 |
emily | things that aren't hardware config should certainly not be in there | 04:37:47 |
emily | so that is more reason to change the generator :P | 04:38:01 |
hexa | if we wanted that to be a default, why not set the default in nixpkgs 😄 | 04:38:12 |
emily | I guess it actually is hardware-y right now | 04:38:29 |
emily | because it iterates over /sys | 04:38:36 |
emily | which your change fixes | 04:38:41 |
uep | i agree but it's some kind of workaround for arguments about the default | 04:38:52 |
uep | (the details of which I forget) | 04:39:03 |
emily | that's also another reason to change it because enumerating only the interfaces that exist at installation to enable DHCP is... | 04:39:04 |
emily | just... | 04:39:09 |
ElvishJerricco | Ok, well if we move it to configuration.nix, we have a different problem that also encourages keeping the abstraction: That file is meant to be the user's introduction to NixOS configuration. It should not contain scary implementation details. | 04:39:29 |