Re: [PATCH] cygwin: include sys/types.h in sys/xattr.h

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

On Jun  3 03:39, Yaakov Selkowitz wrote:
> Using libattr's <xattr/xattr.h> requires consumers to explicitly include
> <sys/types.h> first, but glibc's header in sys/ already contains the include.
> 
> Signed-off-by: Yaakov Selkowitz <yselkowi@redhat.com>
> ---
>  winsup/cygwin/include/sys/xattr.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/winsup/cygwin/include/sys/xattr.h b/winsup/cygwin/include/sys/xattr.h
> index 1f32392..902bb86 100644
> --- a/winsup/cygwin/include/sys/xattr.h
> +++ b/winsup/cygwin/include/sys/xattr.h
> @@ -13,6 +13,7 @@ details. */
>  #ifndef _SYS_XATTR_H
>  #define _SYS_XATTR_H
>  
> +#include <sys/types.h>
>  #include <attr/xattr.h>
>  
>  #endif /* _SYS_XATTR_H */

ACK, please apply.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: signature.asc
Description: PGP signature