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.

Sign up for GitHub

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

Jump to bottom

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

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

wants to merge 1 commit into from
+10 −10

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Jan 4, 2025 •
edited by bedevere-app bot
Loading

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

@erlend-aasland
pythongh-127614: Correctly check for ttyname_r() in configure
f25911a 
PR python#14868 replaced the ttyname() call with ttyname_r(), but the old
check remained.
@erlend-aasland erlend-aasland added skip news needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes labels Jan 4, 2025
@bedevere-app bedevere-app bot mentioned this pull request Jan 4, 2025
Open
@erlend-aasland erlend-aasland linked an issue Jan 4, 2025 that may be closed by this pull request
configure is checking for ttyname but code is using ttyname_r #127614
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@corona10 corona10 Awaiting requested review from corona10 corona10 is a code owner

Assignees
No one assigned
Labels
awaiting core review needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

configure is checking for ttyname but code is using ttyname_r
1 participant
@erlend-aasland