[oe] [meta-networking][PATCH] recipes-connectivity/samba: Only rmdir directories that exist
Mike Looijmans
mike.looijmans at topic.nl
Mon Jan 11 18:06:20 UTC 2016
On 11-01-16 19:00, Khem Raj wrote:
>
>> On Jan 11, 2016, at 9:53 AM, Mike Looijmans <mike.looijmans at topic.nl> wrote:
>>
>> Depending on PACKAGECONFIG selection, the /run/samba directory may not
>> have been created. Make the do_install_append handle both situations
>> by checking whether these directories exist before attempting to remove
>> them.
>>
>> This fixes do_install failing with an error like this:
>> rmdir: failed to remove '/.../samba/4.1.12-r0/image/run/samba': No such file or directory
>>
>> Signed-off-by: Mike Looijmans <mike.looijmans at topic.nl>
>> ---
>> meta-networking/recipes-connectivity/samba/samba_4.1.12.bb | 8 ++++++--
>> 1 file changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta-networking/recipes-connectivity/samba/samba_4.1.12.bb b/meta-networking/recipes-connectivity/samba/samba_4.1.12.bb
>> index a51d31f..8e89e49 100644
>> --- a/meta-networking/recipes-connectivity/samba/samba_4.1.12.bb
>> +++ b/meta-networking/recipes-connectivity/samba/samba_4.1.12.bb
>> @@ -104,8 +104,12 @@ EXTRA_OECONF += "--enable-fhs \
>> LDFLAGS += "-Wl,-z,relro,-z,now"
>>
>> do_install_append() {
>> - rmdir --ignore-fail-on-non-empty "${D}/run/samba"
>> - rmdir --ignore-fail-on-non-empty "${D}/run"
>> + if [ -d "${D}/run" ]; then
>> + if [ -d "${D}/run/samba" ]; then
>> + rmdir --ignore-fail-on-non-empty "${D}/run/samba"
>> + fi
>> + rmdir --ignore-fail-on-non-empty "${D}/run"
>> + fi
>
> why don’t we delete /run completely ? it won’t work if package contents are in there anyway
>
That's what I do in a bbappend, just "rm -rf ${D}/run" (and also replace
the non-functional startup script, but that's distro specific), but I
thought that it might serve some purpose for the one who wrote the recipe.
/run is usually volatile, so putting files in there is pointless, right?
More information about the Openembedded-devel
mailing list