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

Drop support for Python versions earlier than 3.8 #1773

Merged
merged 5 commits into from
Mar 6, 2024

Conversation

mdickinson
Copy link
Member

This PR drops support for Python versions older than 3.8.

@@ -276,13 +276,6 @@ def get_long_description():
Operating System :: POSIX :: Linux
Programming Language :: Python
Programming Language :: Python :: 3
Programming Language :: Python :: 3.6
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing these classifiers - they're a pain to maintain and provide little value. Note that we do have a python_requires keyword below giving the minimum supported version of Python.

setup.py Outdated Show resolved Hide resolved
@mdickinson
Copy link
Member Author

Hmm. A slightly odd failure on Python 3.13 / Windows:

LINK : fatal error LNK1104: cannot open file 'python313t.lib'

@mdickinson
Copy link
Member Author

A slightly odd failure on Python 3.13 / Windows:

Looks like this might be the cause: python/cpython#115582

@mdickinson
Copy link
Member Author

I'm not sure what to do about the Windows failure. It should be fixed as soon as Python 3.13.0a5 is released and is available for use by the setup-python GitHub Action. 3.13.0a5 is expected in around a week from now (according to PEP 719).

3.13.0-alpha.4 has a bug that prevents Traits from compiling on Windows, so until alpha.5 is released, we use alpha.3.
@mdickinson
Copy link
Member Author

@corranwebster I think this is ready for review. I've pinned the Python 3.13 version being used for the core tests; we'll need to remove that pin at some point.

I'll need to adjust the PR expected checks to remove the 3.7 versions; I'll do that just before merging.

Copy link
Contributor

@corranwebster corranwebster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@mdickinson mdickinson merged commit 95cfc08 into main Mar 6, 2024
29 checks passed
@mdickinson mdickinson deleted the dev/drop-support-for-python-37 branch March 6, 2024 08:12
mdickinson added a commit that referenced this pull request Mar 26, 2024
This PR fixes some compiler warnings that I noticed in the failed
Windows build in PR #1773. There are two classes of fixes:

- Fix occurrences of `int` used to represent the size of a tuple / list
/ dict. Those should be `Py_ssize_t`
- Fix two functions that treat a mask as having type `int`, when it
should be `unsigned int`. All the actual masks used have type `unsigned
int`.

The warnings themselves are relatively harmless, but silencing them
gives us a better chance of noticing warnings that we _should_ be taking
notice of.

The PR also adds a `-v` flag to the `pip install` commands that install
Traits, so that compiler warnings will always be visible in the build
log. Without the `-v`, we only get to see the warnings for failed
builds.
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