Re: Patch needed for mdutils

Matthias Urlichs (smurf@work.noris.de)
7 Mar 1998 03:12:24 +0100


Hi,

Martin.Schulze@Infodrom.North.DE (Martin Schulze) writes:
> In file included from mdadd.c:25:
> /usr/include/linux/time.h:30: warning: `FD_SET' redefined
> /usr/include/sys/select.h:60: warning: this is the location of the previous definition
>
> Here's the patch I needed to make to compile the package:
>
> --- linux-2.0.33.orig/include/linux/time.h Wed Jan 7 14:44:04 1998
> +++ linux-2.0.33/include/linux/time.h Wed Jan 7 14:44:27 1998
> +#ifndef FD_SETSIZE
> #define FD_SETSIZE __FD_SETSIZE
> +#endif
[...]

The correct thing to do is to NOT include linux/time.h in the first place.

Documentation should state (and probably does, in several places) that
mixing glibc and kernel include files is not generally a good idea.

This is intentional.

-- 
Matthias Urlichs
noris network GmbH

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu