bug#28567: 27.0.50; `directory-files-recursively' signals an error if there is some inaccessible subdirectory

[Lars Ingebrigtsen]

Philipp <p.stephani2@gmail.com> writes:

> mkdir c c/c
> chmod 0000 c/c
> emacs -Q -batch -f toggle-debug-on-error -eval '(directory-files-recursively 
> "/tmp/c" "foo")'
> Debug on Error enabled globally
> Debugger entered--Lisp error: (file-error "Opening directory" "Permission 
> denied" "/tmp/c/c")
>   file-name-all-completions("" "/tmp/c/c")
>   directory-files-recursively("/tmp/c/c" "foo" nil)
>   directory-files-recursively("/tmp/c" "foo")
>   eval((directory-files-recursively "/tmp/c" "foo"))
>   command-line-1(("-f" "toggle-debug-on-error" "-eval" 
> "(directory-files-recursively \"/tmp/c\" \"foo\")"))
>   command-line()
>   normal-top-level()
>
> It's probably fine to have this behavior by default (some users might
> want strict errors when some subdirectory couldn't be listed), but I
> think the function should have an optional argument to control the
> behavior.

Yup.  I've now fixed this on the trunk -- there's now an optional
PREDICATE parameter that can be t for "ignore file-error".

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no