[PATCH 1/4] Include the POSIX dfcntl.h instead of sys/dfcntl.h

Alan Coopersmith alan.coopersmith at oracle.com
Fri Feb 6 19:28:56 PST 2015


On 02/ 6/15 09:29 AM, Emil Velikov wrote:
> The former is part of the POSIX standard, and (unlike the latter) is
> more widely available.
>
> Cc: Adam Jackson <ajax at redhat.com>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>   src/pciaccess_private.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/pciaccess_private.h b/src/pciaccess_private.h
> index 9f4e8f9..2f05b29 100644
> --- a/src/pciaccess_private.h
> +++ b/src/pciaccess_private.h
> @@ -42,7 +42,7 @@
>    * know of any OS we support where this isn't available in a sufficiently
>    * new version, so warn unconditionally.
>    */
> -#include <sys/fcntl.h>
> +#include <fcntl.h>
>
>   #ifndef O_CLOEXEC
>   #warning O_CLOEXEC not available, please upgrade.
>

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

but you have a typo repeated twice in the subject - there's no d in this
header file name.


-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list