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

[Philipp Stephani <p.stephani2@gmail.com>]

Am Mo., 15. Juli 2019 um 11:53 Uhr schrieb Lars Ingebrigtsen <larsi@gnus.org>:
>
> 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".

Thanks!