gador/nixpkgs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

services.hostapd: convert iwd assertion ito an warning (#445760)

Browse Source
This commit is contained in:
Michele Guerini Rocco
2025-09-24 17:36:15 +00:00
committed by GitHub
parent 635c544b6a be8eb7a3ed
commit 42093d86bd
1 changed files with 10 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
nixos/modules/services/networking/hostapd.nix
View File

@@ -1334,26 +1334,21 @@ in
# we check if wirelessInterfaces is empty as that means all interfaces implicit # we check if wirelessInterfaces is empty as that means all interfaces implicit
shouldWarn = wirelessEnabled && (wirelessInterfaces == [ ] || hasInterfaceConflict); shouldWarn = wirelessEnabled && (wirelessInterfaces == [ ] || hasInterfaceConflict);
in in
if shouldWarn then lib.optional shouldWarn ''
[
''
Some wireless interface is configured for both for client and access point mode: Some wireless interface is configured for both for client and access point mode:
this is not allowed. Either specify `networking.wireless.interfaces` and exclude this is not allowed. Either specify `networking.wireless.interfaces` and exclude
those from `services.hostapd.radios` or make sure to not run the `wpa_supplicant` those from `services.hostapd.radios` or make sure to not run the `wpa_supplicant`
and `hostapd` services simultaneously. and `hostapd` services simultaneously.
'' ''
] ++ lib.optional config.networking.wireless.iwd.enable ''
else hostapd and iwd do conflict,
[ ]; use `networking.wireless.enable` in combination with `networking.wireless.interfaces` to avoid it.
'';
assertions = [ assertions = [
{ {
assertion = cfg.radios != { }; assertion = cfg.radios != { };
message = "At least one radio must be configured with hostapd!"; message = "At least one radio must be configured with hostapd!";
} }
{
assertion = !config.networking.wireless.iwd.enable;
message = "hostapd and iwd conflict, use `networking.wireless.enable` in combination with `networking.wireless.interfaces`";
}
] ]
# Radio warnings # Radio warnings
++ (concatLists ( ++ (concatLists (