Re: ...but now

[Mark Kettenis <kettenis at wins dot uva dot nl>]

Ulrich Drepper <drepper@redhat.com> writes:

> I think all fixable problems are handled in the current CVS version.
> I'm running some tests over night and if everything is fine I'll make
> the first 2.2.1 test release tomorrow.  Speak up quickly if you have
> some problem.

More compilation problems for the Hurd.  `incude/netinet/ether.h' now
uses `size_t', but with the Hurd headers `size_t' isn't defined.  The
attached patch syncs the generic definition of `struct ether_addr'
with the definitions used for Linux.  We now get `size_t' as a
side-effect.  The alternative is explicitly including <stddef.h> in
`include/netinet/ether.h'.

Mark


2001-01-05  Mark Kettenis  <kettenis@gnu.org>

	* sysdeps/generic/netinet/if_ether.h: Include <sys/types.h>.
	(struct ether_addr): Declare ether_addr_octet member as
	`u_int8_t'.  Add __attribute__ ((__packed__)).


Index: sysdeps/generic/netinet/if_ether.h
===================================================================
RCS file: /cvs/glibc/libc/sysdeps/generic/netinet/if_ether.h,v
retrieving revision 1.2
diff -u -p -r1.2 if_ether.h
--- sysdeps/generic/netinet/if_ether.h	1997/06/23 21:51:52	1.2
+++ sysdeps/generic/netinet/if_ether.h	2001/01/05 14:35:22
@@ -1,4 +1,4 @@
-/* Copyright (C) 1996, 1997 Free Software Foundation, Inc.
+/* Copyright (C) 1996, 1997, 2001 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
@@ -21,11 +21,13 @@
 #define __NETINET_IF_ETHER_H	1
 #include <features.h>
 
+#include <sys/types.h>
+
 /* This is a name for the 48 bit ethernet address available on many
    systems.  */
 struct ether_addr
 {
-  unsigned char ether_addr_octet[6];
-};
+  u_int8_t ether_addr_octet[6];
+} __attribute__ ((__packed__));
 
 #endif /* netinet/if_ether.h */