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

Add setuptools as build dependency on Python 3.12+ #765

Closed
wants to merge 1 commit into from

Conversation

uranusjr
Copy link

Fix #696. This implements the lowest effort solution, adding setuptools as a build dependency on Python 3.12 and later, which injects distutils automatically back into the environment for
compatibility. I’ll leave it to the maintainers to decide whether it is worthwhile to maintain this separately from Pendulum 3.

Python 3.12 implements PEP 632 to remove distutils from the standard
library, but Pendulum 2.x's build script depends on the module to build
the C extension. This implements the lowest effort solution, adding
setuptools as a build dependency on Python 3.12 and later, which
injects distutils automatically back into the environment for
compatibility.
@yardensachs
Copy link

yes please!

@chenrui333
Copy link

hi can we get a merge on this PR and maybe a release? Thanks! cc @sdispater @Secrus

devraj added a commit to anomaly/lab-python-server that referenced this pull request Nov 22, 2023
pendulum2.x does not support PEP517 and won't build in python3.12, there's a ticket
asking for a release to make it compatible with python 3.12
python-pendulum/pendulum#765

the project maintainers have commented on this ticket to say v2 will no longer get updates
https://github.com/sdispater/pendulum/issues/753\#issuecomment-1739202965

version3 has pre-releases available on github
https://github.com/sdispater/pendulum/releases/tag/3.0.0b1

this commit is now tracking the beta releases to see if there are any issues with
our existing code, and will join back with the stable v3 release as it becomes available

for large portion of it this should address our requirement for python3.12

note that the Dockerfile is update to track patch releases of the python image

REFS #86
@Secrus
Copy link
Collaborator

Secrus commented Dec 17, 2023

Pendulum 3.0 is now released, so this won't be needed anymore

@Secrus Secrus closed this Dec 17, 2023
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.

4 participants