Opened 7 years ago

Closed 7 years ago

#4123 closed bug (fixed)

Control.Concurrent.MVar doesn't need to depend on the Prelude

Reported by: tibbe Owned by:
Priority: high Milestone: 7.0.1
Component: libraries/base Version: 6.13
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: Building GHC failed Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

Control.Concurrent.MVar currently depends on the Prelude, causing an import cycle in the new I/O manager. This patch replaces the dependency on the Prelude with modules in the GHC hierarchy, breaking the cycle.

Attachments (1)

mvar-no-prelude.dpatch (33.6 KB) - added by tibbe 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by tibbe

Attachment: mvar-no-prelude.dpatch added

comment:1 Changed 7 years ago by igloo

Status: newpatch

comment:2 Changed 7 years ago by simonmar

Milestone: 6.14.1
Priority: normalhigh

comment:3 Changed 7 years ago by simonmar

Resolution: fixed
Status: patchclosed

Done. The attached patch depended on a patch by bos that I don't have, so re-recorded as:

Tue Jun 15 11:54:01 BST 2010  Simon Marlow <marlowsd@gmail.com>
  * Don't depend on Prelude (#4123)
Note: See TracTickets for help on using tickets.