summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2017-07-17 15:16:16 +0200
committerThomas Haller <thaller@redhat.com>2017-07-26 18:48:19 +0200
commitf43222d4118fa4c58235f60de0d532531ba1bd8a (patch)
tree25df38c8b5158282a297185a5f41fd1e6e4450b6
parentb77b259ff4bd8aaf99c7eab778323f13f8b52d8b (diff)
downloadNetworkManager-f43222d4118fa4c58235f60de0d532531ba1bd8a.tar.gz
service: don't install dependency for "NetworkManager-wait-online.service" to "network-online.target.wants"
If we install "NetworkManager-wait-online.service" in the "network-online.target.wants" directory, network-online.target always pulls in NetworkManager-wait-online.service. As it was, it could only be disabled by masking the service. Instead, we should enable NetworkManager-wait-online.sevice via systemd's preset. That is already done for Fedora 26 and newer. Note that NetworkManager-wait-online.sevice already has Install.WantedBy. This way, the dependency is created automatically when enabling the service. https://bugzilla.redhat.com/show_bug.cgi?id=1455704 (cherry picked from commit d61eaf2545c819cab17c29c4c13a3a3bd94b38a6)
-rw-r--r--Makefile.am9
-rw-r--r--contrib/fedora/rpm/NetworkManager.spec2
2 files changed, 0 insertions, 11 deletions
diff --git a/Makefile.am b/Makefile.am
index 9c5b6661d3..5c49eab47c 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -33,7 +33,6 @@ dist_dependencies =
dist_configure_check =
install_data_hook =
install_exec_hook =
-install_exec_local =
uninstall_hook =
TESTS =
BUILT_SOURCES =
@@ -3521,12 +3520,6 @@ data/org.freedesktop.NetworkManager.service: $(srcdir)/data/org.freedesktop.Netw
service_DATA += data/org.freedesktop.NetworkManager.service
-install-exec-local-data:
- install -d $(DESTDIR)$(systemdsystemunitdir)/network-online.target.wants
- ln -sf $(systemdsystemunitdir)/NetworkManager-wait-online.service $(DESTDIR)$(systemdsystemunitdir)/network-online.target.wants
-
-install_exec_local += install-exec-local-data
-
endif
examples_DATA += data/server.conf
@@ -4410,8 +4403,6 @@ dist-hook: $(dist_hook)
install-exec-hook: $(install_exec_hook)
-install-exec-local: $(install_exec_local)
-
install-data-hook: $(install_data_hook)
$(mkinstalldirs) -m 0755 $(DESTDIR)$(nmconfdir)/conf.d
$(mkinstalldirs) -m 0755 $(DESTDIR)$(nmconfdir)/system-connections
diff --git a/contrib/fedora/rpm/NetworkManager.spec b/contrib/fedora/rpm/NetworkManager.spec
index acf2ca03fb..56638b4c66 100644
--- a/contrib/fedora/rpm/NetworkManager.spec
+++ b/contrib/fedora/rpm/NetworkManager.spec
@@ -556,8 +556,6 @@ fi
%{systemd_dir}/NetworkManager.service
%{systemd_dir}/NetworkManager-wait-online.service
%{systemd_dir}/NetworkManager-dispatcher.service
-%dir %{systemd_dir}/network-online.target.wants
-%{systemd_dir}/network-online.target.wants/NetworkManager-wait-online.service
%dir %{_datadir}/doc/NetworkManager/examples
%{_datadir}/doc/NetworkManager/examples/server.conf
%doc NEWS AUTHORS README CONTRIBUTING TODO