Skip to content

gh-125843: fix test_curses.test_attributes on x86-64 macOS - #134252

Merged
picnixz merged 8 commits into
python:mainfrom
picnixz:patch/curses-checks-125843
May 20, 2025
Merged

gh-125843: fix test_curses.test_attributes on x86-64 macOS#134252
picnixz merged 8 commits into
python:mainfrom
picnixz:patch/curses-checks-125843

Conversation

@picnixz

@picnixzpicnixz commented May 19, 2025

Copy link
Copy Markdown
Member

Apparently, on x86-64 MacOS Intel with and without GIL, test_curses.test_attributes failed, but I don't really know why after looking at the code. So, I'll just check by hand to check what kind of error is actually being returned.

@picnixz

This comment was marked as resolved.

@bedevere-bot

This comment was marked as resolved.

@picnixzpicnixz changed the title gh-125843: improve traceback when reporting unexpected curses errorsgh-125843: fix test_curses.test_attributes on x86-64 macOSMay 19, 2025
@picnixz

This comment was marked as resolved.

@bedevere-bot

This comment was marked as resolved.

@picnixz

Copy link
Copy Markdown
MemberAuthor

!buildbot x86-64 MacOS Intel

@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @picnixz for commit 8fcdb53 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F134252%2Fmerge

The command will test the builders whose names match following regular expression: x86-64 MacOS Intel

The builders matched are:

  • x86-64 MacOS Intel ASAN NoGIL PR
  • x86-64 MacOS Intel NoGIL PR

@picnixzpicnixz added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 19, 2025
@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 19, 2025
@picnixzpicnixz added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 19, 2025
@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 19, 2025
@pythonpython deleted a comment from bedevere-botMay 19, 2025
@pythonpython deleted a comment from bedevere-botMay 19, 2025
@picnixzpicnixz added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 19, 2025
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @picnixz for commit 8004b3d 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F134252%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label May 19, 2025
Comment threadModules/_cursesmodule.c Outdated
@picnixz
picnixz enabled auto-merge (squash) May 20, 2025 08:26
@picnixz
picnixz merged commit e007e62 into python:mainMay 20, 2025
@encukou

Copy link
Copy Markdown
Member

Thank you for the fix!

@picnixz
picnixz deleted the patch/curses-checks-125843 branch May 20, 2025 10:09
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
…ython#134252)
While some `libcurses` functions are meant to return OK on success,
this is not always the case for all implementations. As such, we relax
the checks on the return values and allow any non-ERR value to be
considered equivalent to OK.
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
…ython#134252)
While some `libcurses` functions are meant to return OK on success,
this is not always the case for all implementations. As such, we relax
the checks on the return values and allow any non-ERR value to be
considered equivalent to OK.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@picnixz@bedevere-bot@encukou