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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove distutils use #505

Open
jobovy opened this issue Sep 6, 2022 · 1 comment
Open

Remove distutils use #505

jobovy opened this issue Sep 6, 2022 · 1 comment
Assignees
@jobovy
Labels
enhancement pinned

Comments

@jobovy
Copy link
Owner

jobovy commented Sep 6, 2022 •
edited

馃悰 Bug

distutils is deprecated, so we need to replace/remove all of it's use in the code. Currently used in

  • setup.py
  • __intelcompiler.py
  • galpy/util/_load_extension_libs.py
@jobovy jobovy self-assigned this Sep 6, 2022
jobovy added a commit that referenced this issue Sep 6, 2022
@jobovy
distutils.sysconfig as sysconfig --> sysconfig (see #505)
22fd93c
jobovy added a commit that referenced this issue Sep 6, 2022
@jobovy
Replace non-compiler use of distutils with setuptools (see #505)
9ef3593
@jobovy
Copy link
Owner Author

jobovy commented Sep 6, 2022

So far, have gotten rid of all distutils usage except for the compiler stuff in setup.py and __intelcompiler.py. My sense from reading setuptools discussions about this, is that it could be a while until there are decent replacements for that.

Leaving open for now. Note that while distutils will be removed from the standard library in Python 3.12 (Oct 2023), setuptools will probably ship its version that imports the same for longer, so this isn't a major and urgent issue.

@jobovy jobovy added the pinned label Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jobovy jobovy

Labels
enhancement pinned
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jobovy