Subject: bug#66150: 30.0.50; Duplicate :tag in remote-file-name-inhibit-cache

From: "Basil L. Contovounesios" <[email-masked]>
Date: Fri, 22 Sep 2023 15:19:35 +0200

close 66150 29.1
quit

Eli Zaretskii [2023-09-22 14:37 +0300] wrote:

>> Cc: Michael Albinus <[email-masked]>
>> Date: Fri, 22 Sep 2023 11:41:32 +0200
>> From:  "Basil L. Contovounesios" via "Bug reports for GNU Emacs,
>>  the Swiss army knife of text editors" <[email-masked]>
>> 
>> -	  (const   :tag "Do not inhibit file name cache" nil)
>> -	  (const   :tag "Do not use file name cache" t)
>> -	  (integer :tag "Do not use file name cache"
>> -		   :format "Do not use file name cache older then %v seconds"
>> +          (const   :tag "Inhibit remote file cache" t)
>> +          (const   :tag "Cache remote file attributes" nil)
>> +          (integer :tag "Cache remote file attributes with expiration"
>> +                   :format "Cache expiry in seconds: %v"
>>  		   :value 10)))
>
> Let's be consistent.  Either (a) the t value should have the tag of
> "Do not cache remote file attributes", or (b) the nil and the integer
> values should have the tag "Do not inhibit remote file cache" and
> "Inhibit file cache if expired", respectively.  OK?

Of course, thanks.  I went with option (a) for both
remote-file-name-inhibit-cache and
shell-highlight-undef-remote-file-name-inhibit-cache.

Installed on emacs-29, and closing.

Improve remote-file-name-inhibit-cache :type
e27ec0e4144 2023-09-22 15:15:36 +0200
https://git.sv.gnu.org/cgit/emacs.git/commit/?id=e27ec0e4144

-- 
Basil