-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spell doesn't load in Windows #4
Comments
Hey, thanks for yours! I didn't have access to a windows system when I was building that, so I'm Cheers Jonas On 25 February 2016 at 06:30, negm [email protected] wrote:
|
I have changed the source code to work on windows. Because tar can be a little bit finicky, I switched to an LZMA zip file instead of the bzipped one and just used the standard ZipFile module. An advantage to this is that the file takes a little less space and is more cross-platform. I'll make a pull request for the updated version, but if you want to keep yours the way it is I can make a windows fork. |
Hi, I submitted a pull request to fix this issue. I found that a LZMA formatted file doesn't seem to work in a default install of windows. |
@foobarmus many thanks for your work on this, but I kindly have a few questions:
Please any help will be greatly appreciated on how to solve this.
Regards |
I traced the issue to utils.py -> with closing(t.extractfile(tar_path)) as f:
it seems the tarfile library doesn't behave as intended on Windows. I extracted the file manually and changed the code to open it using os.open for now but I intended to make a pull request with a more general solution soon
Thanks for your effort
The text was updated successfully, but these errors were encountered: