[pacman-dev] [PATCH] Allow makepkg to use busybox find

[Allan McRae]

On 04/05/12 09:12, Jeremy Huntwork wrote:
> On 5/3/12 6:16 AM, Dave Reisner wrote:
>>>       if check_option "emptydirs" "n"; then
>>>           msg2 "$(gettext "Removing empty directories...")"
>>> -        find . -depth -type d -empty -delete
>>> +        find . -depth -mindepth 1 -type d -exec rmdir
>>> --ignore-fail-on-non-empty '{}' +
>>
>> There's no way --ignore-fail-on-non-empty is anything but a ridiculous
>> GNU option that busybox picked up.
> 
> I have a solution for the compression of man pages that I believe
> incorporates all the suggestions you gave and works well. It's certainly
> an improvement of the last version. I'll submit that shortly.
> 
> For replacing the -empty param, the most elegant thing I've found is:
> 
> find . -mindepth 1 -depth -type d -exec rmdir "{}" + 2>/dev/null || true
> 
> Anything else is a lot more code and more cumbersome. If the || true in
> this instance is unacceptable, then I will probably just submit the
> changes for the compression section and maintain this modification as a
> private patch or sed command for myself.
> 

Why do you need "|| true"?