commit:     e42ae854f8818adf804c76b193d0524fac4dd1d4
Author:     Nicholas Vinson <nvinson234 <AT> gmail <DOT> com>
AuthorDate: Sun Oct 23 17:10:49 2016 +0000
Commit:     Patrice Clement <monsieurp <AT> gentoo <DOT> org>
CommitDate: Wed Oct 26 21:16:28 2016 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e42ae854

net-firewall/nftables: update systemd file target.

Change the systemd unit files to use

    Before=network-pre.target
    Wants=network-pre.target

as this better aligns with systemd documentation for this type of
service.  Special thanks to Shaun Bouckaert for triggering this review.
He originally asked (via email) about how the connman systemd unit file
handled its dependencies which prompted me to check this package as
well.  As for connman, v1.33-r1 seems to be aligned with systemd
documentation.

Package-Manager: portage-2.3.2
Closes: https://github.com/gentoo/gentoo/pull/2649

Signed-off-by: Patrice Clement <monsieurp <AT> gentoo.org>

 net-firewall/nftables/files/systemd/nftables-restore.service          | 4 ++--
 .../nftables/{nftables-0.6-r3.ebuild => nftables-0.6-r4.ebuild}       | 0
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/net-firewall/nftables/files/systemd/nftables-restore.service 
b/net-firewall/nftables/files/systemd/nftables-restore.service
index fbd5a83..4b68b0a 100644
--- a/net-firewall/nftables/files/systemd/nftables-restore.service
+++ b/net-firewall/nftables/files/systemd/nftables-restore.service
@@ -1,8 +1,8 @@
 [Unit]
 Description=Store and restore nftables firewall rules
 ConditionPathExists=/var/lib/nftables/rules-save
-Before=network.target
-Before=shutdown.target
+Before=network-pre.target
+Wants=network-pre.target
 
 [Service]
 Type=oneshot

diff --git a/net-firewall/nftables/nftables-0.6-r3.ebuild 
b/net-firewall/nftables/nftables-0.6-r4.ebuild
similarity index 100%
rename from net-firewall/nftables/nftables-0.6-r3.ebuild
rename to net-firewall/nftables/nftables-0.6-r4.ebuild

Reply via email to