Re: (interrup-process ...) broken on NT native
>>>> "Gunnar" == Gunnar Evermann
<ge204(a)eng.cam.ac.uk> writes:
Gunnar> Adrian Aichner <aichner(a)ecf.teradyne.com> writes:
> I can't figure out which change broke (interrupt-process ...)
in
> XEmacs 21.1 and XEmacs 21.2 built natively on NT4SP5.
>
> This is the reason why kill-compilation is no longer working in
> *compile* buffers (needless to say: It used to work to
> approx. 21.1.7).
>
> Perhaps you'll have some ideas. I am copying Gunnar who has also done
> work in the process area.
Gunnar> OK, thanks. I am afraid I have followed xemacs-beta very
^ NOT?
Gunnar> closely in the last months.
> This doesn't:
> (start-process "shell" nil "cmd" "/c" "sleep"
"60")
> (message "before: %S" (get-process "shell"))
> (interrupt-process "shell" t)
> (message "after: %S" (get-process "shell"))
Gunnar> could you elaborate what "doesn't work" means? Does it
Oops, sorry. It does neither kill the shell, nor the sub-process.
A sleep process not run as a sub-process of a "cmd" shell can be
killed fine either by name or object.
Gunnar> make a difference whether you use the name or the process?
No.
Gunnar> Have you tried running this in a debugger and single
Gunnar> stepping through process_send_signal()? Alternatively you
Yes. It gave me a bit of a head-ache :-)
SIGINT under NT is implemented by sending CTRL_C_EVENT.
I got lost during debug trying to figure out what's wrong.
Gunnar> could try backing out patches until it works :-(
Sounds pretty scary without your help. Who knows what interactions
exist between your patches and Bens' and others'.
> It would be great if we could fix this for 21.1 even, if not for
> 21.1.10.
Gunnar> indeed. I think I never intended my changes to go into
Gunnar> 21.1, for which I had a simpler version, but this should
Gunnar> still work.
Hmmh, can you give me some pointers to problematic portions of your
patches or point me to the simpler patch on xemacs-patches?
Gunnar> Gunnar
Thanks in advance,
Adrian