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

Remove deprecated distutils for python 3.12 compatability #11893

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

borntohonk
Copy link
Contributor

@borntohonk borntohonk commented Dec 14, 2024

  • PR is based on the DEVELOP branch
  • Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
  • Read the contribution guide

As seen in the medusa discord, distutils causes crashes for end users because of lines 6 and 45 in medusa/updater/update_manager.py

due to references to distutils, which is deprecated in python 3.12

I've also attempted to update helper libraries which are not needed to address the crash itself.

@borntohonk borntohonk force-pushed the develop branch 3 times, most recently from 252a0c3 to efb2783 Compare December 14, 2024 08:26
@borntohonk
Copy link
Contributor Author

(i apologize for messy PR, but I wanted I wanted to address tox after realizing the tests were failing due to test method being deprecated)

@borntohonk
Copy link
Contributor Author

My understanding is that https://github.com/pymedusa/Medusa/blob/master/.github/workflows/api-tests.yml
This errors per the action logs:
ModuleNotFoundError: No module named 'packaging'
I'm not sure if this is addressed with:
pip install packaging

@borntohonk
Copy link
Contributor Author

after a lot of confusion on my end, it seems the api test is using dredd, which is made by apriary, which is owned by oracle since 8 years ago and deprecated last month

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.

1 participant