Re: tramp (2.7.0-pre master/463cd87f0590970cfd2918ba7028304e0f712001); tramp-signal-process doesn't accept string arguments

[Philipp Stephani]

> Am 29.11.2023 um 20:47 schrieb Michael Albinus <michael.albinus@gmx.de>:
> 
> Philipp <p.stephani2@gmail.com> writes:
> 
> Hi Philipp,
> 
>> The function `tramp-signal-process' should accept a process name as a
>> string (to be converted to a process via `get-process') or a process ID
>> converted to a string, as documented for `signal-process'.
> 
> Yep. What about the appended patch?

Almost, but one case is still missing.  Try:

emacs -Q -batch -eval '(progn (require (quote tramp)) (let ((proc 
(start-process "sleep" nil "sleep" "60"))) (signal-process (number-to-string 
(process-id proc)) (quote ABRT))))'

This will fail even with the patch.
`signal-process' accepts a process ID converted to a string.  This doesn't seem 
to be documented, but is necessary for interactive use because `signal-process' 
uses `read-string' to read its first argument interactively.