bug#36831: enhance 'directory not empty' message

[Paul Eggert]

On 7/29/19 1:28 AM, Assaf Gordon wrote:
> +      if (rename_errno == ENOTEMPTY || rename_errno == EEXIST)
> +        {
> +          error (0, 0, _("cannot move %s to %s: Target directory not empty"),
> +                 quoteaf_n (0, src_name), quoteaf_n (1, dst_name));

Although this is an improvement, it is not general enough, as other 
errno values are relevant only for the destination. Better would be to 
have a special case for errno values that matter only for the 
destination, and use the existing code for errno values where we don't 
know whether the problem is the source or the destination. Something 
like the attached, say.

mv.diff
Description: Text Data