Opened 9 years ago

Closed 8 years ago

#5223 closed feature request (fixed)

Make interruptProcessGroupOf interrupt the current process group

Reported by: Favonia Owned by: simonmar
Priority: high Milestone: 7.4.1
Component: libraries/process Version: 7.0.3
Keywords: interrupt process group Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

Make interruptProcessGroupOf interrupt the current process group of the target process under POSIX, instead of assuming the argument is the lead process. This requires the new API getProcessGroupIDOf in the unix library. (See #3994 for more information for interruptProcessGroupOf and #5167 for getProcessGroupIDOf.)

Attachments (1)

0002-Use-current-process-group-to-interrupt-processes.patch (1.8 KB) - added by Favonia 9 years ago.
patch to the package process (v1)

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by Favonia

patch to the package process (v1)

comment:1 Changed 9 years ago by simonmar

Milestone: 7.2.1
Owner: changed from Favonia to simonmar
Priority: normalhigh

comment:2 Changed 9 years ago by simonmar

this looks like it will need some #ifdefs and/or build-depends constraints,

comment:3 Changed 9 years ago by Favonia

Indeed. Well, in my opinion, this is just a subtle technical update, and so no need to hurry. AFAIK we can just wait until the unix package is released under a new version, and then change the dependency requirement in the .cabal file of the process package.

comment:4 in reply to:  3 ; Changed 9 years ago by Favonia

Replying to Favonia:

Indeed. Well, in my opinion, this is just a subtle technical update, and so no need to hurry. AFAIK we can just wait until the unix package is released under a new version, and then change the dependency requirement in the .cabal file of the process package.

I meant that maybe we could apply the patch until the new unix package comes out.

comment:5 in reply to:  4 Changed 9 years ago by Favonia

Replying to Favonia:

I meant that maybe we could apply the patch until the new unix package comes out.

s/until/when/. Sorry for my bad English.

comment:6 Changed 9 years ago by simonmar

Milestone: 7.2.17.4.1

We could bump the unix package version now and add the necessary #ifdefs, but I don't mind either way. I'll bump this ticket to the next milestone anyway.

comment:7 Changed 8 years ago by simonmar

Resolution: fixed
Status: newclosed

Done:

commit fdd02c6464cd73b15195f166377675c9aecf052e
Author: Simon Marlow <marlowsd@gmail.com>
Date:   Mon Nov 14 15:47:21 2011 +0000

    Make interruptProcessGroupOf interrupt the current process group (#5223)
Note: See TracTickets for help on using tickets.