Re: [OE-core] [PATCH v2] mdadm: install the systemd service through Makefile

2019-05-22 Thread Liwei Song
Please ignore this one.

Because the v1 patch has been applied, so this one is not applicable now,
will send it together with another patch to cover the systemd related change.

Thanks,
Liwei.



On 05/21/2019 04:54 PM, Liwei Song wrote:
> The original mdadm service may include variable like BINDIR,
> It should use the real value of it, since the install method
> "install-systemd" in Makefile can easily do such work, so don't
> install it in bb file manually, use "make install-systemd" to
> install all the service of mdadm.
> 
> Signed-off-by: Liwei Song 
> ---
>  meta/recipes-extended/mdadm/mdadm_4.1.bb | 8 +---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/meta/recipes-extended/mdadm/mdadm_4.1.bb 
> b/meta/recipes-extended/mdadm/mdadm_4.1.bb
> index ef5ddf55db39..8ca42a89ddc5 100644
> --- a/meta/recipes-extended/mdadm/mdadm_4.1.bb
> +++ b/meta/recipes-extended/mdadm/mdadm_4.1.bb
> @@ -57,11 +57,11 @@ do_install() {
>  do_install_append() {
>  install -d ${D}/${sysconfdir}/
>  install -m 644 ${S}/mdadm.conf-example ${D}${sysconfdir}/mdadm.conf
> -install -d ${D}/${systemd_unitdir}/system
> -install -m 644 ${WORKDIR}/mdmonitor.service 
> ${D}/${systemd_unitdir}/system
> -install -m 644 ${S}/systemd/mdmon@.service 
> ${D}/${systemd_unitdir}/system
>  install -d ${D}/${sysconfdir}/init.d
>  install -m 755 ${WORKDIR}/mdadm.init 
> ${D}${sysconfdir}/init.d/mdmonitor
> +if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true', 'false', 
> d)}; then
> +   oe_runmake install-systemd DESTDIR=${D}
> +fi
>  }
>  
>  do_compile_ptest() {
> @@ -90,3 +90,5 @@ RRECOMMENDS_${PN}-ptest += " \
>  kernel-module-raid10 \
>  kernel-module-raid456 \
>  "
> +
> +FILES_${PN} += 
> "${@bb.utils.contains('DISTRO_FEATURES','systemd','/lib/systemd/*', '', d)}"
> 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2] mdadm: install the systemd service through Makefile

2019-05-21 Thread Liwei Song
The original mdadm service may include variable like BINDIR,
It should use the real value of it, since the install method
"install-systemd" in Makefile can easily do such work, so don't
install it in bb file manually, use "make install-systemd" to
install all the service of mdadm.

Signed-off-by: Liwei Song 
---
 meta/recipes-extended/mdadm/mdadm_4.1.bb | 8 +---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-extended/mdadm/mdadm_4.1.bb 
b/meta/recipes-extended/mdadm/mdadm_4.1.bb
index ef5ddf55db39..8ca42a89ddc5 100644
--- a/meta/recipes-extended/mdadm/mdadm_4.1.bb
+++ b/meta/recipes-extended/mdadm/mdadm_4.1.bb
@@ -57,11 +57,11 @@ do_install() {
 do_install_append() {
 install -d ${D}/${sysconfdir}/
 install -m 644 ${S}/mdadm.conf-example ${D}${sysconfdir}/mdadm.conf
-install -d ${D}/${systemd_unitdir}/system
-install -m 644 ${WORKDIR}/mdmonitor.service 
${D}/${systemd_unitdir}/system
-install -m 644 ${S}/systemd/mdmon@.service 
${D}/${systemd_unitdir}/system
 install -d ${D}/${sysconfdir}/init.d
 install -m 755 ${WORKDIR}/mdadm.init ${D}${sysconfdir}/init.d/mdmonitor
+if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true', 'false', 
d)}; then
+   oe_runmake install-systemd DESTDIR=${D}
+fi
 }
 
 do_compile_ptest() {
@@ -90,3 +90,5 @@ RRECOMMENDS_${PN}-ptest += " \
 kernel-module-raid10 \
 kernel-module-raid456 \
 "
+
+FILES_${PN} += 
"${@bb.utils.contains('DISTRO_FEATURES','systemd','/lib/systemd/*', '', d)}"
-- 
2.7.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core