Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

cloudscraper exact version requirement no longer working? #170

Closed
ghost opened this issue Apr 2, 2021 · 6 comments
Closed

cloudscraper exact version requirement no longer working? #170

ghost opened this issue Apr 2, 2021 · 6 comments

Comments

@ghost
Copy link

ghost commented Apr 2, 2021

It seems to me that the exact version listen in setup.py is no longer working. I kept getting the error

Failed to find addon version number for game version: GameVersion.retail, https://www.curseforge.com/wow/addons/details

for all my addons. After changing the version requirement for cloudscraper to >= 1.2.46 and updating cloudscraper to 1.2.56 it started working again.

@Mayo77
Copy link

Mayo77 commented Apr 4, 2021

Same issue. but i still cant get mine working. How did you update ckloudscraper?

@ghost
Copy link
Author

ghost commented Apr 5, 2021

I applied the following changes to setup.py and then updated all of those modules. I suspect just updating cloudscraper is enough though, but I stopped testing after I got it working as I was busy at the time.

diff --git a/setup.py b/setup.py
index 518c125..7744a65 100644
--- a/setup.py
+++ b/setup.py
@@ -55,16 +55,16 @@ setup(
         ]
     },
     install_requires=[
-        "beautifulsoup4==4.9.1",
+        "beautifulsoup4>=4.9.1",
         "certifi==2020.6.20",
         "chardet==3.0.4",
-        "cloudscraper==1.2.46",
+        "cloudscraper>=1.2.46",
         "idna==2.10; python_version >= '2.7' and python_version not in '3.0, 3.1, 3.2, 3.3'",
         "pyparsing==2.4.7; python_version >= '2.6' and python_version not in '3.0, 3.1, 3.2, 3.3'",
         "requests==2.24.0",
         "requests-toolbelt==0.9.1",
-        "soupsieve==2.0.1; python_version >= '3.5'",
-        "urllib3==1.25.10; python_version >= '2.7' and python_version not in '3.0, 3.1, 3.2, 3.3, 3.4' and python_version < '4'",
+        "soupsieve>=2.0.1; python_version >= '3.5'",
+        "urllib3>=1.25.10; python_version >= '2.7' and python_version not in '3.0, 3.1, 3.2, 3.3, 3.4' and python_version < '4'",
     ],
     python_requires=">=3.8.0",
 )

@grrttedwards
Copy link
Owner

Hey, thank you for reporting. It looks like my automated tests also failed today, so I think this is going to hit everyone.

I'll experiment and try to push a new release tonight.

@ghost
Copy link
Author

ghost commented Apr 5, 2021

I'm not super familiar with the setup.py install_requirements, is it possible to ignore the bugfix part of the versions somehow? Perhaps something along the lines of cloudscraper>=1.2.0 and cloudscraper < 1.3.0.

That way these small fixes in the upstream modules can just be updated in the environment of the user without needing any changes on the wow-addon-updater side. Minor version changes can still be manually reviewed should they be needed.

@grrttedwards
Copy link
Owner

grrttedwards commented Apr 7, 2021

I try not to actually maintain the setup.py dependencies directly, and use a module that takes the pipenv lock file and transforms it into setup.py dependencies. After updating the lock file you can update it with pipenv-setup sync -d. Anyways, I have put this branch up here: #172 but unfortunately wowace is having issues now, and the tests are failing...

@grrttedwards
Copy link
Owner

Published Release 1.8.1, thanks all

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

No branches or pull requests

2 participants