unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#50190: dired-chmod-program is a revenant
@ 2021-08-24 18:11 Paul Eggert
  2021-08-25 11:42 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Paul Eggert @ 2021-08-24 18:11 UTC (permalink / raw)
  To: 50190

This is a followup from Bug#11912.

The defcustom dired-chmod-program is no longer used by dired or any 
derivative. It is now used only by ange-ftp-process-file.

I suggest marking dired-chmod-program as obsolete (or removing it), and 
changing ange-ftp-process-file's (or (bound-and-true-p 
dired-chmod-program) "chmod") to simply "chmod".





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#50190: dired-chmod-program is a revenant
  2021-08-24 18:11 bug#50190: dired-chmod-program is a revenant Paul Eggert
@ 2021-08-25 11:42 ` Lars Ingebrigtsen
  2021-08-25 17:15   ` Michael Albinus
  0 siblings, 1 reply; 4+ messages in thread
From: Lars Ingebrigtsen @ 2021-08-25 11:42 UTC (permalink / raw)
  To: Paul Eggert; +Cc: 50190, Michael Albinus

Paul Eggert <eggert@cs.ucla.edu> writes:

> The defcustom dired-chmod-program is no longer used by dired or any
> derivative. It is now used only by ange-ftp-process-file.
>
> I suggest marking dired-chmod-program as obsolete (or removing it),
> and changing ange-ftp-process-file's (or (bound-and-true-p
> dired-chmod-program) "chmod") to simply "chmod".

Obsoleting it sounds like a good idea to me.  Perhaps Michael has some
comments on the ange-ftp change; added to the CCs.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#50190: dired-chmod-program is a revenant
  2021-08-25 11:42 ` Lars Ingebrigtsen
@ 2021-08-25 17:15   ` Michael Albinus
  2021-08-26 14:13     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Michael Albinus @ 2021-08-25 17:15 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 50190, Paul Eggert

Lars Ingebrigtsen <larsi@gnus.org> writes:

Hi,

>> The defcustom dired-chmod-program is no longer used by dired or any
>> derivative. It is now used only by ange-ftp-process-file.
>>
>> I suggest marking dired-chmod-program as obsolete (or removing it),
>> and changing ange-ftp-process-file's (or (bound-and-true-p
>> dired-chmod-program) "chmod") to simply "chmod".
>
> Obsoleting it sounds like a good idea to me.  Perhaps Michael has some
> comments on the ange-ftp change; added to the CCs.

I don't know whether somebody uses dired-chmod-program in
ange-ftp. Obsoleting it would be OK; let's see who protests.

Best regards, Michael.





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#50190: dired-chmod-program is a revenant
  2021-08-25 17:15   ` Michael Albinus
@ 2021-08-26 14:13     ` Lars Ingebrigtsen
  0 siblings, 0 replies; 4+ messages in thread
From: Lars Ingebrigtsen @ 2021-08-26 14:13 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 50190, Paul Eggert

Michael Albinus <michael.albinus@gmx.de> writes:

>>> The defcustom dired-chmod-program is no longer used by dired or any
>>> derivative. It is now used only by ange-ftp-process-file.
>>>
>>> I suggest marking dired-chmod-program as obsolete (or removing it),
>>> and changing ange-ftp-process-file's (or (bound-and-true-p
>>> dired-chmod-program) "chmod") to simply "chmod".
>>
>> Obsoleting it sounds like a good idea to me.  Perhaps Michael has some
>> comments on the ange-ftp change; added to the CCs.
>
> I don't know whether somebody uses dired-chmod-program in
> ange-ftp. Obsoleting it would be OK; let's see who protests.

OK; I've now made the variable obsolete (and removed the one usage).  I
did not remove the comments in wdired about it, since Paul's patch in the
other bug report does that.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-08-26 14:13 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-24 18:11 bug#50190: dired-chmod-program is a revenant Paul Eggert
2021-08-25 11:42 ` Lars Ingebrigtsen
2021-08-25 17:15   ` Michael Albinus
2021-08-26 14:13     ` Lars Ingebrigtsen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).