<div dir="ltr"><div style>We found the idea of improving c:ls/1 worthwhile.</div><div style><br></div><div style>Therefore, we will not simply reject the patch.</div><div style><br></div><div style>Instead we will use the implementation in the following</div>
<div style>commit:</div><div style><br></div><div style>  <a href="https://github.com/bjorng/otp/commit/67d83e15fdee068217df7f14a32fb901d7d4d91b">https://github.com/bjorng/otp/commit/67d83e15fdee068217df7f14a32fb901d7d4d91b</a><br>
</div><div><br></div><div style>in which I have credited Bengt for the idea and test case.</div><div style><br></div><div style>We will test it a few days in our daily builds before merging</div><div style>it to maint.</div>
<div style><br></div><div style>/Bjorn</div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 20, 2013 at 4:40 PM, Bengt Kleberg <span dir="ltr"><<a href="mailto:bengt.kleberg@ericsson.com" target="_blank">bengt.kleberg@ericsson.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I do not know why file:list_dir/1 has failed with {error, enoent}.<br>
Or to be precise, I do not know what failure with {error, enoent} means<br>
for the different cases I get from directories, files, links, special<br>
<div class="im HOEnZb">files, none-existing files and the combination of read/no-read access<br>
for these.<br>
<br>
<br>
</div><span class="HOEnZb"><font color="#888888">bengt<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Mon, 2013-05-20 at 16:36 +0200, Björn Gustavsson wrote:<br>
> I'll try to clarify my question:<br>
><br>
><br>
><br>
> Why test for existence at all if<br>
> file:list_dir/1 has already told you that the<br>
> file exists?<br>
><br>
><br>
> On Mon, May 20, 2013 at 4:22 PM, Bengt Kleberg<br>
> <<a href="mailto:bengt.kleberg@ericsson.com">bengt.kleberg@ericsson.com</a>> wrote:<br>
><br>
><br>
>         The filelib:is_dir/is_file/is_regular functions confuse me. I<br>
>         am unsure<br>
>         what they will return for directories, files, links, special<br>
>         files,<br>
>         none-existing files and the combination of read/no-read access<br>
>         for<br>
>         these.<br>
><br>
>         file:read_link_info/1 is more predictable.<br>
><br>
><br>
>         bengt<br>
><br>
>         On Mon, 2013-05-20 at 15:59 +0200, Björn Gustavsson wrote:<br>
>         > On Mon, May 20, 2013 at 1:45 PM, Bengt Kleberg<br>
>         > <<a href="mailto:bengt.kleberg@ericsson.com">bengt.kleberg@ericsson.com</a>> wrote:<br>
>         >         6> file:list_dir("nosuchfile").<br>
>         >         {error,enotdir}<br>
>         ><br>
>         >         I think to use file:read_link_info/1 to establish<br>
>         the<br>
>         >         existence of the<br>
>         >         file/directory.<br>
>         ><br>
>         > Why?<br>
>         ><br>
>         ><br>
>         > If file:list_dir(Name) returns {error,enotdir}, it means<br>
>         that<br>
>         > there exists a file named Name (but it is not a directory).<br>
>         ><br>
>         ><br>
>         > If no such file exists, the return value will be<br>
>         {error,enoent}.<br>
>         ><br>
>         ><br>
>         > In what situation would it be useful to call<br>
>         > file:read_link_info/1?<br>
>         ><br>
>         ><br>
>         ><br>
>         > --<br>
>         > Björn Gustavsson, Erlang/OTP, Ericsson AB<br>
><br>
><br>
><br>
><br>
><br>
><br>
> --<br>
> Björn Gustavsson, Erlang/OTP, Ericsson AB<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div>