Opened 9 years ago

Closed 9 years ago

#5167 closed feature request (fixed)

New POSIX process group API

Reported by: Favonia Owned by: simonmar
Priority: high Milestone: 7.2.1
Component: libraries/unix Version: 7.0.3
Keywords: 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

API changes to Posix.Process in order to have two new features:

  1. A different type for POSIX process group ID from POSIX process ID
  1. Access to full power of POSIX process group API

Please see the following page for more information: POSIX process group API reform proposal

Attachments (4)

0001-Make-ProcessGroupID-a-newtype-of-CPid.patch (1.6 KB) - added by Favonia 9 years ago.
patch to the package base
0001-Change-the-POSIX-process-group-API.-trac-5167.2.patch (5.9 KB) - added by Favonia 9 years ago.
patch to the package unix
0002-Test-cases-for-the-new-process-group-API.patch (2.3 KB) - added by Favonia 9 years ago.
test cases for the package unix (version 3)
0001-Change-the-POSIX-process-group-API.-trac-5167.patch (4.5 KB) - added by Favonia 9 years ago.
patch to the package unix (version 3)

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by Favonia

patch to the package base

comment:1 in reply to:  description Changed 9 years ago by Favonia

Component: libraries/processlibraries/unix

Discussion on the libraries list has just started. No consensus has reached yet and this ticket is more like a place to hold patches and test cases. Please make any comments on the libraries list, thanks.

Changed 9 years ago by Favonia

patch to the package unix

Changed 9 years ago by Favonia

test cases for the package unix (version 3)

Changed 9 years ago by Favonia

patch to the package unix (version 3)

comment:3 Changed 9 years ago by Favonia

Status: newpatch

The discussion thread has survived for two weeks on the mailing list. The patches have been modified according to Simon's suggestion except the name newProcess[For]. Please review attachment:0001-Change-the-POSIX-process-group-API.-trac-5167.patch attachment:0002-Test-cases-for-the-new-process-group-API.patch (and ignore other patch files).

PS: I need help in the documentation since I do not speak English. Please feel free to modify/improve the descriptions of functions.

comment:4 Changed 9 years ago by simonmar

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

comment:5 Changed 9 years ago by simonmar

Resolution: fixed
Status: patchclosed

Committed, thanks!

changeset:ff80c44c25ab6233fdd0180275c9981400007c1c changeset:6f88d3dd6e107c996b701b7f4fccaef171440722

(these links won't work until we import multiple repos into the GHC Trac)

Note: See TracTickets for help on using tickets.