Skip to content

[3.13] gh-155974: Add a test for the window attributes after a failed write - #156566

Merged
serhiy-storchaka merged 1 commit into
python:3.13from
serhiy-storchaka:guard-155974-3.13
Aug 29, 2026
Merged

[3.13] gh-155974: Add a test for the window attributes after a failed write#156566
serhiy-storchaka merged 1 commit into
python:3.13from
serhiy-storchaka:guard-155974-3.13

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Aug 29, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

A write with an attr restores the window rendition afterwards, whether it succeeded or failed. This branch does the right thing already -- the regression fixed by #155975 was introduced in 3.15 by 30dde1e, which is not here -- so this only adds the test as a guard.

The test reads the rendition by writing a cell with it and reading that cell back with inch(), since window.getattrs() is 3.16 only.

@serhiy-storchaka
pythongh-155974: Add a test for the window attributes after a failed …
6c0d271 
…write

The bug was introduced and fixed in 3.15, but the behaviour is worth
guarding here too.
@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label Aug 29, 2026
@bedevere-app bedevere-app Bot mentioned this pull request Aug 29, 2026
Closed
@serhiy-storchaka
serhiy-storchaka merged commit 71b65fc into python:3.13 Aug 29, 2026
45 checks passed
@serhiy-storchaka
serhiy-storchaka deleted the guard-155974-3.13 branch August 29, 2026 15:34
@bedevere-bot

bedevere-bot commented Aug 29, 2026

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot s390x Fedora Stable 3.13 (tier-3) has failed when building commit 71b65fc.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/1658/builds/1119) and take a look at the build logs.
  4. Check if the failure is related to this commit (71b65fc) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/1658/builds/1119

Summary of the results of the build (if available):

Click to see traceback logs 
Note: switching to '71b65fcb95f7464139dd28b2d9d6736d61eba926'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 71b65fcb95f74 [3.13] gh-155974: Add a test for the window attributes after a failed write (GH-156566)
Switched to and reset branch '3.13'

In file included from ./Include/Python.h:14,
                 from ./Modules/_testexternalinspection.c:53:
./pyconfig.h:1985:9: warning: ‘_POSIX_C_SOURCE’ redefined
 1985 | #define _POSIX_C_SOURCE 200809L
      |         ^~~~~~~~~~~~~~~
In file included from /usr/include/bits/libc-header-start.h:33,
                 from /usr/include/stdint.h:26,
                 from /usr/lib/gcc/s390x-redhat-linux/16/include/stdint.h:11,
                 from /usr/include/elf.h:24,
                 from ./Modules/_testexternalinspection.c:4:
/usr/include/features.h:319:10: note: this is the location of the previous definition
  319 | # define _POSIX_C_SOURCE        202405L
      |          ^~~~~~~~~~~~~~~
./pyconfig.h:2006:9: warning: ‘_XOPEN_SOURCE’ redefined
 2006 | #define _XOPEN_SOURCE 700
      |         ^~~~~~~~~~~~~
/usr/include/features.h:234:10: note: this is the location of the previous definition
  234 | # define _XOPEN_SOURCE  800
      |          ^~~~~~~~~~~~~
ar: unable to copy file 'libpython3.13d.a'; reason: No space left on device
make: *** [Makefile:1065: libpython3.13d.a] Error 1

@bedevere-bot

bedevere-bot commented Aug 29, 2026

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot s390x Fedora Stable Clang 3.13 (tier-3) has failed when building commit 71b65fc.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/1667/builds/1109) and take a look at the build logs.
  4. Check if the failure is related to this commit (71b65fc) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/1667/builds/1109

Summary of the results of the build (if available):

Click to see traceback logs 
Note: switching to '71b65fcb95f7464139dd28b2d9d6736d61eba926'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 71b65fcb95f74 [3.13] gh-155974: Add a test for the window attributes after a failed write (GH-156566)
Switched to and reset branch '3.13'

configure: WARNING: s390x-ibm-linux-gnu/clang is not supported
configure: WARNING:

Platform "s390x-ibm-linux-gnu" with compiler "clang" is not supported by the
CPython core team, see https://peps.python.org/pep-0011/ for more information.


In file included from ./Modules/_testexternalinspection.c:53:
In file included from ./Include/Python.h:14:
./pyconfig.h:1985:9: warning: '_POSIX_C_SOURCE' macro redefined [-Wmacro-redefined]
 1985 | #define _POSIX_C_SOURCE 200809L
      |         ^
/usr/include/features.h:319:10: note: previous definition is here
  319 | # define _POSIX_C_SOURCE        202405L
      |          ^
In file included from ./Modules/_testexternalinspection.c:53:
In file included from ./Include/Python.h:14:
./pyconfig.h:2006:9: warning: '_XOPEN_SOURCE' macro redefined [-Wmacro-redefined]
 2006 | #define _XOPEN_SOURCE 700
      |         ^
/usr/include/features.h:234:10: note: previous definition is here
  234 | # define _XOPEN_SOURCE  800
      |          ^
2 warnings generated.
ar: libpython3.13d.a: error reading Objects/capsule.o: No space left on device
make: *** [Makefile:1065: libpython3.13d.a] Error 1

@bedevere-bot

bedevere-bot commented Aug 29, 2026

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot s390x Fedora Stable Clang Installed 3.13 (tier-3) has failed when building commit 71b65fc.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/1656/builds/1107) and take a look at the build logs.
  4. Check if the failure is related to this commit (71b65fc) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/1656/builds/1107

Summary of the results of the build (if available):

Click to see traceback logs 
Note: switching to '71b65fcb95f7464139dd28b2d9d6736d61eba926'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 71b65fcb95f74 [3.13] gh-155974: Add a test for the window attributes after a failed write (GH-156566)
Switched to and reset branch '3.13'

configure: WARNING: s390x-ibm-linux-gnu/clang is not supported
configure: WARNING:

Platform "s390x-ibm-linux-gnu" with compiler "clang" is not supported by the
CPython core team, see https://peps.python.org/pep-0011/ for more information.


rror: unable to open output file 'Objects/typevarobject.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/typevarobject.o] Error 1
make: *** Waiting for unfinished jobs....
rror: unable to open output file 'Objects/setobject.o': 'No space left on device'
1 error generated.
rror: unable to open output file 'Objects/tupleobject.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/setobject.o] Error 1
make: *** [Makefile:3042: Objects/tupleobject.o] Error 1
rror: unable to open output file 'Objects/dictobject.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/dictobject.o] Error 1
rror: unable to open output file 'Objects/unicodectype.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/unicodectype.o] Error 1
rror: unable to open output file 'Objects/longobject.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/longobject.o] Error 1
rror: unable to open output file 'Parser/parser.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Parser/parser.o] Error 1
rror: unable to open output file 'Objects/obmalloc.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/obmalloc.o] Error 1
rror: unable to open output file 'Objects/typeobject.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/typeobject.o] Error 1
rror: unable to open output file 'Objects/unicodeobject.o': 'No space left on device'
1 error generated.
make: *** [Makefile:3042: Objects/unicodeobject.o] Error 1

chmod: cannot access 'target/': No such file or directory

find: ‘build’: No such file or directory
find: ‘build’: No such file or directory
find: ‘build’: No such file or directory
find: ‘build’: No such file or directory
make: [Makefile:3132: clean-retain-profile] Error 1 (ignored)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

skip news tests Tests in the Lib/test dir topic-curses

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@serhiy-storchaka @bedevere-bot