[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#29225: Tramp backup-by-copying in a Converting ACL Invalid argument
From: |
Eli Zaretskii |
Subject: |
bug#29225: Tramp backup-by-copying in a Converting ACL Invalid argument error on Windows 7 |
Date: |
Fri, 10 Nov 2017 15:06:49 +0200 |
> From: Michael Albinus <michael.albinus@gmx.de>
> Cc: Shuguang Sun <shuguang@gmail.com>, 29225@debbugs.gnu.org
> Date: Fri, 10 Nov 2017 13:41:04 +0100
>
> "Value is t if setting of ACL was successful, nil otherwise."
>
> The error message "Converting ACL" "Invalid argument" comes from
> Ffile_acl of fileio.c. It *raises* an error instead of silently
> returning Qnil, as advertised by the docstring. A similar wrong
> behaviour I've found in `tramp-smb-handle-set-file-acl'.
>
> I would convert both functions to return nil instead of raising an
> error, in the emacs-26 branch. Any objection?
I think it should return nil when acl_errno_valid returns false, and
otherwise signal an error. It currently calls acl_errno_valid in one
of the two places where error could happen, but not in the other. And
the doc string should be amended to say that. WDYT?