search

CodeConstruct / mctp

MCTP userspace tools
GNU General Public License v2.0
32 stars 19 forks source link

meson: Install all systemd units #13

Open leiyu-bytedance opened 1 year ago

leiyu-bytedance commented 1 year ago

Previously the systemd units were not installed and it depends on the bitbake recipe (in meta-openembedded) to install the service only.

Update the meson.build and install all the units. The bitbake recipe will need to be updated to add missing units to SYSTEMD_SERVICE:${PN}.

jk-ozlabs commented 1 year ago

So these units were a bit of a draft in their current state, and don't (yet!) mean to be a fully-formed system MCTP config.

Does this mean they're working for your environment? Any changes in the infrastructure around these that you would propose?

leiyu-bytedance commented 1 year ago

Yes, to integrate the recipe into BMC and make it useful, we need all the units to be installed. So in the bitbake recipe, additional changes are needed as well:

SYSTEMD_SERVICE:${PN}:append = " \
    mctp.target \
    mctp-local.target \
"