Skip to content

Remove special handling for mac os versions < Mac OS X 10.2 #131090

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

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

maxbachmann
Copy link
Contributor

@maxbachmann maxbachmann commented Mar 11, 2025

Mac OS X 10.1 isn't supported anymore so we can drop this special handling

I don't think this requires an issue and news entry

@vstinner
Copy link
Member

cc @ned-deily @sobolevn

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

OSX 10.2 was released in 2003, I think that it is safe to drop the support. All tests pass on recent macOS (14.5)

» ./python.exe -m test test_socket
Using random seed: 74284401
0:00:00 load avg: 2.36 Run 1 test sequentially in a single process
0:00:00 load avg: 2.36 [1/1] test_socket
/Users/sobolev/Desktop/cpython/Lib/test/test_socket.py:3044: RuntimeWarning: received malformed or improperly-truncated ancillary data
  result = sock.recvmsg_into([buf], *args)
/Users/sobolev/Desktop/cpython/Lib/test/test_socket.py:2953: RuntimeWarning: received malformed or improperly-truncated ancillary data
  result = sock.recvmsg(bufsize, *args)
0:00:27 load avg: 2.04 [1/1] test_socket passed

== Tests result: SUCCESS ==

1 test OK.

Total duration: 27.5 sec
Total tests: run=737 skipped=249
Total test files: run=1/1
Result: SUCCESS

Thank you!

@ned-deily ned-deily merged commit f7ac656 into python:main Mar 13, 2025
51 checks passed
@ned-deily
Copy link
Member

Thanks for the PR, @maxbachmann!

plashchynski pushed a commit to plashchynski/cpython that referenced this pull request Mar 17, 2025
seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants