Remove legacy settings service
xyz.openbmc_project.Settings has replaced this function
Resolves openbmc/openbmc#2339
Change-Id: Ifec6a2fda088c4631e3835d3276a54e09b5d7147
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
diff --git a/common/recipes-phosphor/datetime/phosphor-time-manager.bb b/common/recipes-phosphor/datetime/phosphor-time-manager.bb
index 2a476f0..1e09799 100644
--- a/common/recipes-phosphor/datetime/phosphor-time-manager.bb
+++ b/common/recipes-phosphor/datetime/phosphor-time-manager.bb
@@ -15,7 +15,7 @@
DEPENDS += "sdbusplus-native"
DEPENDS += "phosphor-logging"
DEPENDS += "phosphor-dbus-interfaces"
-RDEPENDS_${PN} += "virtual-obmc-settings-mgmt"
+RDEPENDS_${PN} += "${VIRTUAL-RUNTIME_obmc-settings-mgmt}"
RDEPENDS_${PN} += "network"
RDEPENDS_${PN} += "libmapper"
RDEPENDS_${PN} += "libsystemd"
diff --git a/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb b/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb
index 91ceff9..563ed26 100644
--- a/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb
+++ b/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb
@@ -47,7 +47,8 @@
SYSTEMD_SERVICE_${PN} += "xyz.openbmc_project.Ipmi.Internal.SoftPowerOff.service phosphor-ipmi-host.service"
-RRECOMMENDS_${PN} += "virtual-obmc-settings-mgmt"
+RRECOMMENDS_${PN} += "${VIRTUAL-RUNTIME_obmc-settings-mgmt}"
+
require ${PN}.inc
diff --git a/common/recipes-phosphor/ipmi/phosphor-ipmi-host/phosphor-ipmi-host.service b/common/recipes-phosphor/ipmi/phosphor-ipmi-host/phosphor-ipmi-host.service
index a70b272..3f10e11 100644
--- a/common/recipes-phosphor/ipmi/phosphor-ipmi-host/phosphor-ipmi-host.service
+++ b/common/recipes-phosphor/ipmi/phosphor-ipmi-host/phosphor-ipmi-host.service
@@ -11,9 +11,6 @@
After=mapper-wait@-xyz-openbmc_project-control-host0-restriction_mode.service
Wants=clear-once.service
After=clear-once.service
-# TODO openbmc/openbmc#2008 - remove wants/after on the old settings below
-Wants=mapper-wait@-org-openbmc-settings-host0.service
-After=mapper-wait@-org-openbmc-settings-host0.service
[Service]
Restart=always
diff --git a/common/recipes-phosphor/packagegroups/packagegroup-obmc-apps.bb b/common/recipes-phosphor/packagegroups/packagegroup-obmc-apps.bb
index f014f17..fef72dc 100644
--- a/common/recipes-phosphor/packagegroups/packagegroup-obmc-apps.bb
+++ b/common/recipes-phosphor/packagegroups/packagegroup-obmc-apps.bb
@@ -112,7 +112,7 @@
SUMMARY_${PN}-settings = "Settings applications"
RDEPENDS_${PN}-settings = " \
- ${VIRTUAL-RUNTIME_obmc-settings-mgr} \
+ ${VIRTUAL-RUNTIME_obmc-settings-mgmt} \
"
SUMMARY_${PN}-network = "BMC Network Manager"
diff --git a/common/recipes-phosphor/settings/phosphor-settings.bb b/common/recipes-phosphor/settings/phosphor-settings.bb
deleted file mode 100644
index 8e94d8f..0000000
--- a/common/recipes-phosphor/settings/phosphor-settings.bb
+++ /dev/null
@@ -1,50 +0,0 @@
-SUMMARY = "Settings DBUS object"
-DESCRIPTION = "Settings DBUS object"
-HOMEPAGE = "http://github.com/openbmc/phosphor-settingsd"
-PR = "r1"
-
-inherit allarch
-inherit obmc-phosphor-license
-inherit setuptools
-inherit obmc-phosphor-dbus-service
-inherit pythonnative
-
-include phosphor-settings-manager.inc
-
-DBUS_SERVICE_${PN} = "org.openbmc.settings.Host.service"
-
-DEPENDS += "python-pyyaml-native"
-RDEPENDS_${PN} += "python-dbus python-pygobject python-ipy"
-PROVIDES += "virtual/obmc-settings-mgmt"
-RPROVIDES_${PN} += "virtual-obmc-settings-mgmt"
-
-SRC_URI += "file://merge_settings.py"
-
-S = "${WORKDIR}/git"
-
-# Collect files in SRC_URI that end in ".override.yml" and call a script that
-# writes their contents over that of settings.yaml, which is then updated to
-# the merged data values.
-# This doesn't correctly handle globs in ".override.yml" entries in SRC_URI.
-python do_merge_settings () {
- import subprocess
-
- # TODO: Perform the merge in a temporary directory?
- workdir = d.getVar('WORKDIR', True)
- cmd = []
- cmd.append(os.path.join(workdir, 'merge_settings.py'))
- cmd.append(os.path.join(d.getVar('S', True), 'settings.yaml'))
-
- fetch = bb.fetch2.Fetch([], d)
- override_urls = [url for url in fetch.urls if url.endswith('.override.yml')]
- for url in override_urls:
- bb.debug(2, 'Overriding with source: ' + url)
- local_base = os.path.basename(fetch.localpath(url))
- filename = os.path.join(workdir, local_base)
- cmd.append(filename)
-
- # Invoke the script and don't catch any resulting exception.
- subprocess.check_call(cmd)
-}
-# python-pyyaml-native is installed by do_configure, so put this task after
-addtask merge_settings after do_configure before do_compile
diff --git a/common/recipes-phosphor/settings/phosphor-settings/org.openbmc.settings.Host.service b/common/recipes-phosphor/settings/phosphor-settings/org.openbmc.settings.Host.service
deleted file mode 100644
index fbb64d5..0000000
--- a/common/recipes-phosphor/settings/phosphor-settings/org.openbmc.settings.Host.service
+++ /dev/null
@@ -1,18 +0,0 @@
-[Unit]
-Description=Phosphor Settings Manager
-Before=mapper-wait@-org-openbmc-settings-host0.service
-Wants=obmc-mapper.target
-After=obmc-mapper.target
-
-[Service]
-ExecStart=/usr/bin/env settings_manager.py
-SyslogIdentifier=settings_manager.py
-Environment="PYTHONUNBUFFERED=1"
-Restart=always
-Type=dbus
-BusName={BUSNAME}
-Nice=19
-TimeoutStartSec=180s
-
-[Install]
-WantedBy={SYSTEMD_DEFAULT_TARGET}