Ruslan Ermilov <[EMAIL PROTECTED]> writes:
> I see the same breakage here.
> 
> The breakage has nothing to do with -Werror.  Recent commit by
> Mike Barcroft to sys/*/endian.h is the culprit.
> 
> But the actual problem is with gdb.  After a lot of experimenting
> I've found that contrib/gdb.291/gdb/defs.h includes "nm.h"
> (gnu/usr.bin/binutils/gdb/${MACHINE_ARCH}/nm.h) which are too
> different between i386 and alpha.  In the i386 case, "nm.h"
> includes <sys/types.h> which exposes __BSD_VISIBLE (see how
> these affect the *_ENDIAN macros in sys/*/endian.h).  But not
> in the alpha case.  Applying this patch exposes the same bug
> on i386:
>
[...]

David added <sys/types.h> to src/gnu/usr.bin/binutils/gdb/alpha/nm.h 
19 hours ago, so the problem should no longer exist.  I won't know for
another three hours (when my alpha is finished buildworld).

> The solution is to fix defs.h:
> 
> %%%
> Index: defs.h
> ===================================================================
> RCS file: /home/ncvs/src/contrib/gdb.291/gdb/defs.h,v
> retrieving revision 1.3
> diff -u -p -r1.3 defs.h
> --- defs.h    11 Apr 2001 16:15:19 -0000      1.3
> +++ defs.h    12 Apr 2002 13:53:37 -0000
> @@ -841,6 +841,8 @@ extern void free ();
>  
>  #ifdef HAVE_ENDIAN_H
>  #include <endian.h>
> +#else
> +#include <sys/types.h>
>  #endif
>  
>  #if !defined (BIG_ENDIAN)
> %%%

I think <machine/endian.h> would be better here.  It may not have
worked because of the missing <sys/cdefs.h> issue.

Sorry about the breakage.

Best regards,
Mike Barcroft

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to