Skip to content

gh-157949: Remove dead UnixWare, OpenServer and ReliantUNIX support code - #157950

Merged
vstinner merged 4 commits into
python:mainfrom
vasiliyk:remove-unixware
Nov 5, 2026
Merged

vstinner merged 4 commits into
python:mainfrom
vasiliyk:remove-unixware

Conversation

@vasiliyk

@vasiliyk vasiliyk commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Removes the configure.ac branches for SCO UnixWare 7, OpenUNIX 8, SCO OpenServer 5 and ReliantUNIX, the matching pyconfig.h block, the SCO-only preprocessor branches in the curses and posix modules, and the 'unixware7' platform string in test_fcntl.

Tested on macOS (clang) and on Linux (GCC 14)

@vasiliyk

vasiliyk commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

@vstinner could you take a look when you have a moment?
This remove the dead UnixWare, OpenServer and ReliantUNIX paths; nothing changes for supported platforms.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

There is still a reference to SCO in configure.ac (and pyconfig.h.in):

AC_DEFINE([SYS_SELECT_WITH_SYS_TIME], [1],
  [Define if  you can safely include both <sys/select.h> and <sys/time.h>
   (which you can't on SCO ODT 3.0).])

The comment should also be updated in PC/pyconfig.h (remove (which you can't on SCO ODT 3.0)):

/* Define if  you can safely include both <sys/select.h> and <sys/time.h>
   (which you can't on SCO ODT 3.0). */
/* #undef SYS_SELECT_WITH_SYS_TIME */

@vasiliyk
vasiliyk requested a review from a team as a code owner November 5, 2026 15:00
@vasiliyk

vasiliyk commented Nov 5, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the review! I removed the 'SCO ODT 3.0' notes.

@vstinner
vstinner merged commit ca824a9 into python:main Nov 5, 2026
57 checks passed
@vstinner

vstinner commented Nov 5, 2026

Copy link
Copy Markdown
Member

Merged, thanks for cleaning the old dust!

@vasiliyk
vasiliyk deleted the remove-unixware branch November 5, 2026 17:08
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