patch applied (packages/bytestring): Make sure that we hClose
file handles when we are done with them
simonpj at microsoft.com
Mon Oct 6 06:22:29 EDT 2008
We agree. We're going to ship the latest bytestring release with GHC.
Ian will make a mega-patch to make the GHC bytestring fork align with the release. That makes the short term pain easy (just pull), but is not a long term strategy. But we're changing GHC's version control system story soon anyway (at which point we'll explicitly depend only on released versions of libraries like bytestring), so we it doesn't have to be a long term strategy.
| -----Original Message-----
| From: cvs-libraries-bounces at haskell.org [mailto:cvs-libraries-bounces at haskell.org] On Behalf
| Of Don Stewart
| Sent: 05 October 2008 19:01
| To: Ian Lynagh
| Cc: cvs-libraries at haskell.org
| Subject: Re: patch applied (packages/bytestring): Make sure that we hClose file handles when
| we are done with them
| > Sun Oct 5 05:11:50 PDT 2008 Ian Lynagh <igloo at earth.li>
| > * Make sure that we hClose file handles when we are done with them
| > M ./Data/ByteString.hs -1 +2
| > M ./Data/ByteString/Lazy.hs -1 +1
| This stuff worries me, ad hoc fixing of bugs already closed upstream.
| And without tests, (when there are already tests in the upstream repo for this
| very behaviour).
| Wed Dec 19 22:06:13 PST 2007 Don Stewart <dons at galois.com>
| * For lazy IO operations, be sure to hClose the resource on EOF
| Wed Dec 19 22:07:36 PST 2007 Don Stewart <dons at galois.com>
| * add test for lazy hclose working
| Meanwhile, there's twelve months of other patches you're not
| merging into the GHC fork.
| I worry that the concerns of the GHC release process don't align with the
| concerns of library writers. We want to ship the best Haskell code, versus
| shipping whatever builds in the GHC tree. There seems to be unclear direction
| about what we're trying to do here.
| Duncan's already sent the win32 dep removal patch already. Now we just need
| GHCi to not use bytestring to do its little bit of IO.
| Then you can update to bytestring 0.9.1.3 released yesterday, specifically for
| the release,
| -- Don
| Cvs-libraries mailing list
| Cvs-libraries at haskell.org
More information about the Cvs-libraries