Re: [PATCH 5/7] kheaders: remove meaningless -R option of 'ls'

From: Joel Fernandes
Date: Tue Jul 02 2019 - 12:54:10 EST


On Mon, Jul 01, 2019 at 09:58:43AM +0900, Masahiro Yamada wrote:
> The -R option of 'ls' is supposed to be used for directories.
>
> -R, --recursive
> list subdirectories recursively
>
> Since 'find ... -type f' only matches to regular files, we do not
> expect directories passed to the 'ls' command here.
>
> Giving -R is harmless at least, but unneeded.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>

Reviewed-by: Joel Fernandes (Google) <joel@xxxxxxxxxxxxxxxxx>