Skip to content
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

Bump psycopg2-binary to support Python 3.11 #220

Closed
wants to merge 1 commit into from

Conversation

StevenBarre
Copy link

In OCP 4.16 openshift/python:latest is now 3.11 instead of 3.8 and no longer builds with the old psycopg2

@frenzymadness
Copy link
Member

Thanks for the PR, but the master branch here contains Django 1.1, which has already been discontinued a long time ago. It might be a good idea to switch to alternative branches, where we do have still supported Django 4.2, for example.

@frenzymadness
Copy link
Member

Another issue is backward compatibility of this old branch with older Pythons.

@StevenBarre
Copy link
Author

ah ok, maybe there needs to be something in the master Readme describing which branches are stable and should be used for what?

Also, perhaps https://github.com/sclorg/django-ex/blob/4.2.x/openshift/templates/django-postgresql-persistent.json#L452 SOURCE_REPOSITORY_REF should be set to the branch name in the branches?

Let me know if you'd like me to open PRs for either of those suggestions!

@frenzymadness
Copy link
Member

There is this note in the readme in the master branch:

NOTE: This is the oldest example of Django application for specific Django LTS version. If you want a newer version, check the alternative branches.

and other branches correspond to the versions of Django LTS releases.

There are many possible improvements because this project is defacto unowned and unmaintained. I fix an issue here and there because I'm using this project for testing in s2i-python-container but any other contributions are more than welcome.

@StevenBarre StevenBarre deleted the StevenBarre-patch-1 branch January 24, 2025 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants