Search
Friday, 15th of September 2017, 0:19:13 UTC
11:16:44
hlavaty
hi on windoes, is there a way to get pid for run-program process, which is suitable for taskkill? sb-ext:process-pid gives a number which doesn't seem to be windows pid
11:20:07
jackdaniel
hlavaty: I believe that it is meant to be killed with sb-ext:process-close called on the return value of sb-ext:run-program
11:23:18
hlavaty
i hope it does kill the whole process tree
11:24:51
hlavaty
actually it looks like sb-ext:process-pid returns handle for which GetProcessID returns the actual pid
11:28:46
scymtym
ACTION thought PROCESS-CLOSE detaches the child process without killing it
11:28:49
hlavaty
sb-ext:process-close doesnt say anything about killing the process
11:29:12
jackdaniel
OK, sorry for confusing things then
11:29:40
scymtym
on non-win32 platforms there is PROCESS-KILL
11:31:24
hlavaty
scymtym: yeah, we are using that, but the issue is on windows
11:32:01
jackdaniel
then maybe (defun terminate-win-process (pid) (sb-ext:run-program "taskkill" '(…))) ;? if pid is correct
11:32:46
hlavaty
jackdaniel: yes, but as i said initially, sb-ext:process-pid doesn't return the pid
11:33:24
hlavaty
we'll try sb-ext:process-pid -> handle -> GetProcessId -> pid -> taskkill and see if it works
11:34:01
scymtym
hlavaty: maybe try this via sb-alien: https://msdn.microsoft.com/en-us/library/ms686714(VS.85).aspx
11:35:17
hlavaty
scymtym: yeah that is an option, although it doesnt seem to advertise killing the whole process subtree
11:38:27
scymtym
neither does PROCESS-KILL
11:39:15
scymtym
unless the signal is SIGTERM or SIGINT and the child process takes care of it itself
11:39:52
hlavaty
sb-ext:process-pid -> GetProcessId -> taskkill seems to work
12:16:08
stassats
the hppa build is broken because it has an ldb instruction, the new instruction format defines functions => conflict
12:16:19
stassats
maybe %ldb plus an instruction macro named ldb
12:17:37
stassats
or there's some other way
12:19:12
dougk
op-encoder-name can add an exception for LDB
Friday, 15th of September 2017, 12:19:13 UTC