-
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
Windows Python 3.8+: Search current folder for discid.dll #144
base: main
Are you sure you want to change the base?
Windows Python 3.8+: Search current folder for discid.dll #144
Conversation
The failed check is the same PyPy3 check that failed with all previous pull requests:
|
Tested successfully on:
|
I put this in draft, leaving some time for the discussion to take place a little bit. And I want to try git magic to queue my IF change behind @omgitsraven change |
Fixes JonnyJD#124 and JonnyJD#143 without breaking other OS nor older Python
07edba3
to
af1f8d7
Compare
I managed to include original bug fix author in the PR. |
I tested 3.0.0-dev and patched 3.0.0-dev on my 2 other environments as well now. |
I leave you manage the version change, if needed. |
I can confirm that this patch works as expected with Windows 10 Pro 64 bit, Python 3.12 and isrcsubmit 2.1.0 (I think I already replaced discid.dll with a 64 bit version a few years ago). |
Fixes #143 (Windows Python 3.8+ Could not find module 'discid.dll'), without tampering with other setups.
This #125 fix from @omgitsraven was NG for Python 3.7 and less, while OK for Python 3.8+.
I found how to test Python version with https://stackoverflow.com/a/3016198/2236179 and I also test if script is running in Windows.
Tested on Linux (condition not met, do the script runs good as usual) and Windows Python 3.10.1 (condition met, fixes #143 and now the script is running good).
The script works in both cases.
I no longer have old python version but I have tested my version condition and it should work to prevent the new line in python 3.7 and earlier.
Then something must be done to bump version number and to publish new version zip files.
PATCH
If you want to test in your environment, the patch is to insert the following code, before the first
try
of the script:TEST