Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

zip module does not have close method #73

Open
kriszyp opened this issue Nov 18, 2009 · 3 comments
Open

zip module does not have close method #73

kriszyp opened this issue Nov 18, 2009 · 3 comments

Comments

@kriszyp
Copy link

kriszyp commented Nov 18, 2009

The zip module does not have a close method, and so calling unzip leaves the file in an open/locked state. Fix is here:
http://github.com/kriszyp/narwhal/commit/27b49bfe9c5372f820f3130d302606b797f8c24a

@kriskowal
Copy link
Collaborator

Landed in kriskowal/master kriskowal/refactor. I cherry-picked; you're tracking cadorn's stuff, right? Does it work for you?

@kriszyp
Copy link
Author

kriszyp commented Nov 21, 2009

I have pulled in cadorn's master, and unzip still works with it. Or are you saying that cadorn's master should fix unzip to close? I don't see anything in cadorn's zip.js that would close.

@kriskowal
Copy link
Collaborator

No, unrelated; tusk does not work for me on cadorn/experimental and was wondering if anyone else had succeeded in using it. So it would seem; I'll have to look deeper.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants