Bug report

Bug description:

Minor nit, but there is a check for the definition of EX_OK before sysexits.h is included. Meaning it is always redefined when that header exists.

  In file included from ./Modules/posixmodule.c:295:
  /usr/include/sysexits.h:92:9: warning: "EX_OK" redefined
     92 | #define EX_OK           0       /* successful termination */
        |         ^~~~~ 
  ./Modules/posixmodule.c:56:11: note: this is the location of the previous definition
     56 | #  define EX_OK EXIT_SUCCESS
      |           ^~~~~ 

The C standard says redefinition's like this aren't allowed, but in practice I've only seen compilers warn about them.

CPython versions tested on:

CPython main branch

Operating systems tested on:

Linux

Linked PRs

• gh-129539: Include sysexits.h before checking EX_OK #129540
• gh-129539: Reorganize posixmodule.c header #129558
• [3.13] gh-129539: Include sysexits.h before checking EX_OK #129590
• [3.12] gh-129539: Include sysexits.h before checking EX_OK (#129590) #129609