https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96168

Tim Turner <timturnerc at yahoo dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |timturnerc at yahoo dot com

--- Comment #13 from Tim Turner <timturnerc at yahoo dot com> ---
amd64-linux-siginfo.c: Adjust include order to avoid gnulib error 
    http://www.compilatori.com/
    On Fedora rawhide, after updating to glibc-2.33, I'm seeing the
    following build failure:
    http://www.acpirateradio.co.uk/
      CXX    nat/amd64-linux-siginfo.o
    In file included from /usr/include/bits/sigstksz.h:24,
http://www.logoarts.co.uk/  
                     from /usr/include/signal.h:315,
                     from ../gnulib/import/signal.h:52,
http://www.slipstone.co.uk/  
                     from /ironwood1/sourceware-git/rawhide-gnulib
http://www.mconstantine.co.uk
/bld/../../worktree-gnulib/gdbserver/../gdb/nat/amd64-linux-siginfo.c:20:
http://embermanchester.uk/  
    ../gnulib/import/unistd.h:663:3: error: #error "Please include config.h
first."
      663 |  #error "Please include config.h first." http://connstr.net/  
          |   ^~~~~

    glibc-2.33 has changed signal.h to now include
http://www.go-mk-websites.co.uk/  <bits/sigstksz.h> which,
    in turn, includes <unistd.h>. http://joerg.li/  For a gdb build, this
causes the gnulib
    version of unistd.h to be pulled in first.  The build failure shown
https://www.mktrade.fi/
    above happens because gnulib's config.h has not been included before
    the include of <signal.h>. http://www.jopspeech.com/ 

    The fix is simple - we just rearrange the order of the header file
    includes to make sure that gdbsupport/ http://fishingnewsletters.co.uk/ 
commondefs.h is included before
    attempting to include signal.h.  Note that gdbsupport/commondefs.h
    includes <gnulib/config.h>. http://www.wearelondonmade.com/ 

    Build and regression tested on Fedora 33.  On Fedora rawhide, GDB
    builds again. https://waytowhatsnext.com/ 

    gdb/ChangeLog:
glibc-2.33 has changed signal.h to now include <bits/sigstksz.h> which,
    in turn, includes <unistd.h>. http://www.iu-bloomington.com/  For a gdb
build, this causes the gnulib
    version of unistd.h to be http://the-hunters.org/  pulled in first.  The
build failure shown
    above happens because gnulib's https://komiya-dental.com/ config.h has not
been included before
    the include of <signal.h>. http://www-look-4.com/ 

    The fix is simple - we just rearrange the order of the header file
https://www.webb-dev.co.uk/ 
    includes to make sure that gdbsupport/commondefs.h is included before
    attempting to include signal.h.  Note that gdbsupport/commondefs.h
    includes <gnulib/config.h>.

Reply via email to