commit | 089a4a34914e0fcfd2b0690c5476c35d4672faaf | [log] [tgz] |
---|---|---|
author | Patrick Williams <patrick@stwcx.xyz> | Sat Apr 19 12:39:38 2025 -0400 |
committer | Patrick Williams <patrick@stwcx.xyz> | Mon Apr 21 11:01:45 2025 -0400 |
tree | f44e3618a4fcdceb7f98d6efe3032a855f3e0f80 | |
parent | 7e446a407a579ec72f48009008ad2f83c01c2262 [diff] |
phosphor-sync-software-manager: move to libexec Non-user executables are suppose to be installed into libexec[1]. [1]: https://github.com/openbmc/docs/blob/master/anti-patterns.md#incorrect-placement-of-executables-in-sbin-usrsbin-or-bin-usrbin Tested: ``` $ meson install -C builddir --destdir /tmp/phosphor-bmc-code-mgmt $ find /tmp/phosphor-bmc-code-mgmt/ -name phosphor-sync-software-manager /tmp/phosphor-bmc-code-mgmt/usr/local/libexec/phosphor-bmc-code-mgmt/phosphor-sync-software-manager ``` Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ib85bbdf919c244f361d7208d988b243c4a8be6a0
diff --git a/bmc/meson.build b/bmc/meson.build index cf7152b..abe18fb 100644 --- a/bmc/meson.build +++ b/bmc/meson.build
@@ -165,6 +165,7 @@ 'sync_watch.cpp', dependencies: deps, install: true, + install_dir: get_option('libexecdir') / 'phosphor-code-mgmt', ) install_data('synclist', install_dir: get_option('sysconfdir'))
diff --git a/bmc/xyz.openbmc_project.Software.Sync.service.in b/bmc/xyz.openbmc_project.Software.Sync.service.in index 5feeaa5..d9db2af 100644 --- a/bmc/xyz.openbmc_project.Software.Sync.service.in +++ b/bmc/xyz.openbmc_project.Software.Sync.service.in
@@ -2,7 +2,7 @@ Description=Phosphor Sync Manager that syncs files to the alternate BMC chip [Service] -ExecStart=/usr/bin/phosphor-sync-software-manager +ExecStart=/usr/libexec/phosphor-code-mgmt/phosphor-sync-software-manager Restart=always [Install]