fcntl.ioctl() regression: mutate_flag now mandatory for read-only buffer #3881

Activity

1. julian-klode

   changed the title [-]fcntl.ioctl() regression[/-] [+]fcntl.ioctl() regression: mutate_flag now mandatory for read-only buffer[/+] on Mar 24, 2020
2. julian-klode commented on Mar 24, 2020

   

   julian-klode

   on Mar 24, 2020

   ContributorAuthor

   More actions

   This also breaks compatibility with Python 2 code, as it does not accept the mutate_flag argument.

   TypeError                                 Traceback (most recent call last)
   <ipython-input-6-73e86fea1fbd> in <module>()
   ----> 1 fcntl.ioctl(2, termios.TIOCGWINSZ, 8 * b' ', True)
   
   TypeError: ioctl requires a file or file descriptor, an integer and optionally an integer or buffer argument
   
3. julian-klode

   added a commit that references this issue on Mar 24, 2020

   fcntl: make mutate_flag optional for ioctl w/ read-only buffer

   ...

   This issue will close once commit 23dc493 is merged into the 'master' branch.

   23dc493
4. julian-klode commented on Mar 24, 2020

   

   julian-klode

   on Mar 24, 2020

   ContributorAuthor

   More actions

   Fix in #3882
5. srittau

   closed this as completedin #3882on Mar 24, 2020
6. srittau

   added a commit that references this issue on Mar 24, 2020

   fcntl: make mutate_flag optional for ioctl w/ read-only buffer (#3882)

   ...

   This issue will close once commit 009b269 is merged into the 'main' branch.

   009b269
7. jfillmore

   added a commit that references this issue on Jul 9, 2021

   Workaround mypy 0.770

   ...

   afed4b2