This PR explicitly pins setuptools<81 in requirements.txt to resolve a fatal crash that occurs when initializing the Django web container on a fresh Docker build.
The Issue:
Recent versions of the setuptools package (82.0.0 and newer) have permanently removed the deprecated pkg_resources module. Because the project's legacy voting libraries (py3votecore and pygraph) rely on this API to declare their namespaces, the web server throws a ModuleNotFoundError: No module named 'pkg_resources' and halts execution. This breaks the out-of-the-box Docker onboarding pipeline.
The Fix:
By pinning setuptools to a version prior to the deprecation purge, the missing API is restored and the web container is allowed to boot cleanly.
Future Considerations:
To safely remove this pin in the future, the upstream math libraries will either need to be updated to use the importlib.metadata standard, or the voting application will need to migrate to actively maintained dependencies.
https://github.com/GrandComicsDatabase/gcd-django/pull/715
(1 file)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.![]()
@gemini-code-assist[bot] commented on this pull request.
This pull request adds a constraint for setuptools<81 to the requirements.txt file. The reviewer noted that setuptools is already declared earlier in the file and recommended updating the existing declaration to avoid duplicate entries and potential dependency resolution issues.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
In requirements.txt:
> @@ -65,3 +65,4 @@ redis<5.1 rq<2.7 django-rq<3.3 git+https://github.com/GrandComicsDatabase/haystack-rqueue.git +setuptools<81
The setuptools package is already declared on line 43 of this file. Declaring the same package multiple times with different constraints can lead to dependency resolution issues or confusion. Please update the existing declaration on line 43 to setuptools<81 and remove this duplicate entry.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.![]()
@jhunterjActual pushed 1 commit.
—
View it on GitHub or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.![]()
Looks good, I also had to do this on my test installation.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.![]()