[PATCH newlib 0/1] sys/signal.h needs sys/_intsup.h

Joel Sherrill joel@rtems.org
Wed Aug 25 19:12:44 GMT 2021


Hi

The recent addition of the sig2str block of code for definitions and 
prototypes resulted in the following one line program not compiling
for RTEMS targets:

#include <sys/signal.h>

Turned out that __STDINT_EXP used to conditionalize the definition
of SIG2STR_MAX isn't defined unless <sys/_intsup.h> is included.
I guess the test code got lucky.

It's a simple patch that needed more background and investigation
than code.

Is it safe to assume that including each POSIX and Standard C Library file
independently should compile? If so, I will file a ticket to at least at
those to the RTEMS compile only tests like the ones we have that check
a method can be used per the specific includes in the POSIX specification.

Sorry this slipped through.

--joel

Joel Sherrill (1):
  sys/signal.h: <sys/_intsup.h> is needed for __STDINT_EXP

 newlib/libc/include/sys/signal.h | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.24.4



More information about the Newlib mailing list