proper hostapd dependencies #100

Merged
merged 2 commits into from May 4, 2013

Conversation

Projects
None yet
4 participants

hostapd needs to be started after bind/dhcpd/nat.

modules/services/networking/hostapd.nix
+
+ path = [ pkgs.hostapd ];
+
+ after = [ "${cfg.interface}-cfg.service" "nat.service" "bind.service" "dhcpd.service"];
@jcumming

jcumming Feb 24, 2013

Hrm. It is possible that someone would want to run an Access Point that doesn't provide DNS, DHCP, or a network uplink. Perhaps these should be linked into configuration options somehow?

@jcumming

jcumming Feb 24, 2013

Also, we don't need 'bind', we need a DNS resolver. 'unbound' would work admirably, I'm sure.

@edolstra

edolstra Feb 25, 2013

Owner

An "after" dependency does not make hostapd require those services, it just says that if they exist, hostapd should be started after them.

Member

Phreedom commented May 3, 2013

I assume this is tested and works. It looks good to me. Any objections to merging?

@ghost ghost assigned Phreedom May 3, 2013

jcumming commented May 4, 2013

I've been running this for about a month without problems.

domenkozar added a commit that referenced this pull request May 4, 2013

@domenkozar domenkozar merged commit 01887f2 into NixOS:master May 4, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment