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

[Mike Looijmans]

Definitely, once the RDEPENDS is there, people will forget about it and it 
will still be there long after we've forgotten why it was put there...

On 18-05-18 15:20, Burton, Ross wrote:
> 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
>>>>
>>>> --
>>>> 

Kind regards,

Mike Looijmans
System Expert

TOPIC Products
Materiaalweg 4, NL-5681 RJ Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: mike.looijmans at topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail



_______________________________________________
>>>> 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
>>