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 python 3.8 support #1008

Merged
merged 5 commits into from
Dec 12, 2024
Merged

Drop python 3.8 support #1008

merged 5 commits into from
Dec 12, 2024

Conversation

qkaiser
Copy link
Contributor

@qkaiser qkaiser commented Nov 15, 2024

Python 3.8 reached EOL on 2024-10-07. Our dependencies will likely drop support for that version as well, so it will be a burden to keep-up support for it sooner or later.

@qkaiser qkaiser self-assigned this Nov 15, 2024
@qkaiser qkaiser marked this pull request as draft November 15, 2024 09:00
@qkaiser
Copy link
Contributor Author

qkaiser commented Nov 15, 2024

Draft until we're ready to flip the switch.

@qkaiser qkaiser force-pushed the drop-python3.8 branch 2 times, most recently from 169eeb0 to ee4a2ca Compare November 22, 2024 11:13
@qkaiser qkaiser linked an issue Nov 22, 2024 that may be closed by this pull request
@qkaiser qkaiser force-pushed the drop-python3.8 branch 2 times, most recently from 0bc041a to 07105bb Compare November 23, 2024 11:12
@qkaiser qkaiser marked this pull request as ready for review November 23, 2024 11:38
@qkaiser
Copy link
Contributor Author

qkaiser commented Nov 23, 2024

Once reviewed it can be merged. The support drop will reach end users only when we do a tagged release :)

@qkaiser
Copy link
Contributor Author

qkaiser commented Nov 23, 2024

I replaced the required checks that was set to Python 3.8 tests to Python 3.9 tests in our repo config.

@qkaiser qkaiser added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file python Pull requests that update Python code CI/CD Pull request that updates our Github CI/CD labels Nov 29, 2024
@qkaiser qkaiser requested a review from e3krisztian December 4, 2024 08:37
@qkaiser qkaiser mentioned this pull request Dec 4, 2024
Python 3.8 reached EOL on 2024-10-07. Our dependencies will likely drop
support for that version as well, so it will be a burden to keep-up
support for it sooner or later.
Remove the cstruct dependency requirements relaxation introduced by
94f9129 to maintain Python 3.8 support.

It's no longer needed so now we use cstruct version 4.0 and up.
We cannot use Path.readlink to check if the symlink target is
not set (that happens with f_badsymlinks.img on Darwin systems, you need
a broken filesystem and an OS that supports it).

That's because Path.readlink returns a Path object with an empty name
(i.e. Path("")), which is indiscernable from a symlink target that is
the current directory (i.e. Path(".")). Since there is no difference
between Path("") and Path("."), we can't differentiate between a symlink
with an empty target and a symlink that points at the current directory.
@qkaiser qkaiser merged commit 2b946b0 into main Dec 12, 2024
14 checks passed
@qkaiser qkaiser deleted the drop-python3.8 branch December 12, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Pull request that updates our Github CI/CD dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dropping support for Python 3.8 at the end of the year
2 participants