Re: Conduit missing closing file handles

Tom Ellis <[email protected]> Fri, 1 May 2026 18:10:41 +0100
Newsgroups gmane.comp.lang.haskell.cafe
Message-ID <afTekceVijx-Wjrz@cloudinit-builder>
On Fri, May 01, 2026 at 05:38:00PM +0200, Henning Thielemann wrote:
> On Fri, 1 May 2026, Tom Ellis wrote:
> 
> > On Fri, May 01, 2026 at 04:53:33PM +0200, Henning Thielemann wrote:
> > 
> > > Ok great, I tried and now I get an error from tar-conduit:
> > > 
> > > TarCreationError "<tarPayload>: Too much payload (32752) for file with size
> > > (412): avmedia/res/av02048.png"
> > > 
> > Thanks for trying it.  Do you have some code you can share that shows
> > how to plug it all together into an executable?
> 
> This is fileInfoFromZipEntry [3] after applying it to the options userName
> userId groupName groupId attrSrc.

Thanks, I fixed the bug. You can get the fixed version from:

https://github.com/tomjaguarpaw/ad/blob/353fd275123a1ca50c7981349ec25c3997d4a18e/BluefinZipTar/BluefinZipTar.hs

Please let me know if it works.  (I included a main for testing
purposes, but you still just need convertZipToTar and its
dependencies.)

Tom


PS

The bug was that I was using Data.Conduit.List.isolate:

https://hackage-content.haskell.org/package/conduit-1.3.6.1/docs/Data-Conduit-List.html#v:isolate

but I should have used Data.Conduit.Binary.isolate:

https://hackage-content.haskell.org/package/conduit-extra-1.3.8/docs/Data-Conduit-Binary.html#v:isolate

Fixed in this commit:

https://github.com/tomjaguarpaw/ad/commit/15f179e6827da5b58123f1dbb0f6542a6f7e613d
_______________________________________________
Haskell-Cafe mailing list -- [email protected]
To (un)subscribe, modify options or view archives go to:
Only members subscribed via the mailman list are allowed to post.