Skip to content

gh-127614: Correctly check for ttyname_r() in configure - #128503

Merged
erlend-aasland merged 1 commit into
python:mainfrom
erlend-aasland:autoconf/detect-ttyname_r
Jan 7, 2025
Merged

erlend-aasland merged 1 commit into
python:mainfrom
erlend-aasland:autoconf/detect-ttyname_r

Conversation

@erlend-aasland

@erlend-aasland erlend-aasland commented Jan 4, 2025 •

Copy link
Copy Markdown
Contributor

PR #14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.

PR python#14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.
@miss-islington-app

This comment was marked as outdated.

@miss-islington-app

This comment was marked as outdated.

@miss-islington-app

This comment was marked as outdated.

@bedevere-app

bedevere-app Bot commented Jan 7, 2025

Copy link
Copy Markdown

GH-128598 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 only security fixes label Jan 7, 2025
erlend-aasland added a commit to erlend-aasland/cpython that referenced this pull request Jan 7, 2025
…ython#128503)

(cherry picked from commit e08b282)

PR python#14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.
@bedevere-app

bedevere-app Bot commented Jan 7, 2025

Copy link
Copy Markdown

GH-128599 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Jan 7, 2025
erlend-aasland added a commit that referenced this pull request Jan 7, 2025
…) (#128599)

(cherry picked from commit e08b282)

PR #14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
…#128503)

PR python#14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.
erlend-aasland added a commit that referenced this pull request Jan 11, 2025
…) (#128598)

(cherry picked from commit e08b282)

PR #14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.
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.

configure is checking for ttyname but code is using ttyname_r

1 participant