Let me chime in here as maintainer of NM in Debian: The reason why we ship the patch, which removes Wants/Before=network.target is, that this lead to dependency cycles if services/sysv init scripts in rcS (sysinit.target) had Should-Start/Required-Start: $network, like for example /etc/init.d/rpcbind in Debian.
If there is no such sysv init script Ubuntu, this patch can be dropped indeed.
As for Colin's remark in #25, sounds sensible, to add After=network.target to wpa_supplicant.service as well.
Have you tried adding that and see if that fixes the problem on shutdown with WiFi?
Let me chime in here as maintainer of NM in Debian: The reason why we ship the patch, which removes Wants/Before= network. target is, that this lead to dependency cycles if services/sysv init scripts in rcS (sysinit.target) had Should- Start/Required- Start: $network, like for example /etc/init.d/rpcbind in Debian.
If there is no such sysv init script Ubuntu, this patch can be dropped indeed.
As for Colin's remark in #25, sounds sensible, to add After=network. target to wpa_supplicant. service as well.
Have you tried adding that and see if that fixes the problem on shutdown with WiFi?