[OE-core] [PATCH] systemd: Add util-linux-umount to RDEPENDS

[Burton, Ross]

Fixing busybox is a better solution. yes.

Ross

On 18 May 2018 at 13:24, Fabio Berton <fabio.berton at ossystems.com.br> wrote:
> Yes, I tested this patch seems to work.
>
> But bug report has:
>
> "Yeah, this appears to be a limitation of busybox, and needs to be fixed in
> busybox. We focus on util-linux proper, which has supported this option
> since generations"
>
> So, I add umount from util-linux because systemd focus on util-linux and
> systemd recipe already has util-linux-mount.
>
> If add patch to busybox is a better solution I can do this and send to list.
>
> Fabio
>
> On Thu, May 17, 2018 at 4:53 PM, Andre McCurdy <armccurdy at gmail.com> wrote:
>>
>> On Thu, May 17, 2018 at 12:15 PM, Fabio Berton
>> <fabio.berton at ossystems.com.br> wrote:
>> > Images with systemd fails on reboot with error:
>> >
>> >    [FAILED] Failed unmounting Temporary Directory (/tmp).
>> >
>> > This happens because busybox umount command doesn't has -c
>> > option. Add util-linux-umount that has this option to fix
>> > this issue.
>> >
>> > See details: https://github.com/systemd/systemd/issues/7786
>>
>> That bug report mentions a fix to busybox. Since the fix is available
>> in upstream busybox, maybe better to just backport the patch to our
>> version of buysbox:
>>
>>
>> https://git.busybox.net/busybox/commit/?id=426134128112738c97a665170b21153ef0764b7d
>>
>> > Signed-off-by: Fabio Berton <fabio.berton at ossystems.com.br>
>> > ---
>> >  meta/recipes-core/systemd/systemd_237.bb | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/meta/recipes-core/systemd/systemd_237.bb
>> > b/meta/recipes-core/systemd/systemd_237.bb
>> > index b7c2113255f..3b0f4580d49 100644
>> > --- a/meta/recipes-core/systemd/systemd_237.bb
>> > +++ b/meta/recipes-core/systemd/systemd_237.bb
>> > @@ -478,7 +478,7 @@ FILES_${PN} = " ${base_bindir}/* \
>> >
>> >  FILES_${PN}-dev += "${base_libdir}/security/*.la
>> > ${datadir}/dbus-1/interfaces/ ${sysconfdir}/rpm/macros.systemd"
>> >
>> > -RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (= ${EXTENDPKGV})
>> > util-linux-agetty"
>> > +RDEPENDS_${PN} += "kmod dbus util-linux-mount util-linux-umount udev (=
>> > ${EXTENDPKGV}) util-linux-agetty"
>> >  RDEPENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG',
>> > 'serial-getty-generator', '', 'systemd-serialgetty', d)}"
>> >  RDEPENDS_${PN} += "volatile-binds update-rc.d"
>> >
>> > --
>> > 2.14.2
>> >
>> > --
>> > _______________________________________________
>> > Openembedded-core mailing list
>> > Openembedded-core at lists.openembedded.org
>> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>