-
Notifications
You must be signed in to change notification settings - Fork 13
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
doesn't work with 64 bit python on windows (only 32 bit libdiscid dll) #73
Comments
I did a bit of reading and it looks like even when using ctypes, this is all in one process. So the problem is that a 64 bit Python can't use a 32 bit DLL. This is the same on all platforms, but on Linux there are no binary builds of libdiscid (distributions compile for the needed architecture) and on Mac there are universal binaries (ppa, intel32, intel64). |
Hello @JonnyJD, I am under Windows 7 64 bits and I don't have any errors. * I am stuck at « ERROR: Invalid credentials: caused by: HTTP Error 401: Unauthorized » but it may be due to my Firewall (#114). |
@JonnyJD I will use your script again now. |
Hello @JonnyJD now I realise my mistake. |
I downloaded 64 bit discid.dll from https://ci.appveyor.com/project/MetaBrainz/libdiscid/build/artifacts and overwritten the file and it's working OK! |
I’m thinking to drop (shipped) 32-bit support starting from 3.x. Several Linux distributions have dropped/are dropping 32-bit support. Microsoft stopped supporting new 32-bit installs last summer (though still supporting existing ones). For systems where you make the program yourself, this should be a non-issue since Python doesn’t care, but it will be slightly less complexity for me as maintainer. Esp. for platforms I don’t have access to. I’ll try and see if there’s an easy fix for 2.1.1 (thanks for PR @jesus2099!), and if so, 2.1.1 will have both 32-bit and 64-bit support… and will probably be the last official build version to be shipped with 32-bit support. |
My laptop has been re-installed and I cannot use this any more.
I don't remember how to fix it:
Update: this is fixed with #144 |
I am now on a new Windows 10 Pro and if I don't manually replace
This PR #127 is merged but the downloadable zip are still the same old ones. |
Using 64 bit Python 3 on (64 bit) Windows 8:
Removing 64 bit Python and installing Python 3.3 works (after fixing #71).
Reported by Stalker-X on IRC.
The text was updated successfully, but these errors were encountered: