[Bug libc/21560] sys/wait.h signal.h namespace

["cvs-commit at gcc dot gnu.org" <sourceware-bugzilla at sourceware dot org>]

https://sourceware.org/bugzilla/show_bug.cgi?id=21560

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  0862cdcdc814185345ab3e6eda4bb1b81efdf76a (commit)
      from  b4971123e9e681126cb8eaae6d5de56be84c9e98 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=0862cdcdc814185345ab3e6eda4bb1b81efdf76a

commit 0862cdcdc814185345ab3e6eda4bb1b81efdf76a
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Fri Jun 9 13:45:37 2017 +0000

    Condition signal.h inclusion in sys/wait.h (bug 21560).

    sys/wait.h includes signal.h unconditionally.  But the permission to
    do so is UX-shaded in XPG4.2, and XSI-shaded in POSIX before 2008, so
    this should not be unconditional.  This patch fixes this
    conservatively: the include is kept, but conditioned on the standards
    that permit it (meaning it is still present by default, because
    non-XSI POSIX.1:2008 is enabled by default).  <bits/types.h> is now
    included unconditionally to provide the required definition of
    __pid_t; it was previously included via <signal.h>.  Some standards
    require pid_t to be defined here, and all allow it to be defined here;
    previously defined via <signal.h>, it's now defined directly in this
    header.

    Tested for x86_64.  This does not fix any of the sys/wait.h
    conformtest failures, but substantially reduces the number of
    namespace failures for sys/wait.h for XPG4 and POSIX.

        [BZ #21560]
        * posix/sys/wait.h: Condition include of <signal.h> on
        [__USE_XOPEN_EXTENDED || __USE_XOPEN2K8].  Include <bits/types.h>
        unconditionally.
        [!__pid_t_defined] (pid_t): Define typedef.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog        |    8 ++++++++
 posix/sys/wait.h |   11 +++++++++--
 2 files changed, 17 insertions(+), 2 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.