[PATCH 3/4] Include config.h before any other headers.

Alan Coopersmith alan.coopersmith at oracle.com
Sat Feb 7 11:19:48 PST 2015


On 02/ 7/15 10:16 AM, Emil Velikov wrote:
> On Saturday, 7 February 2015, Alan Coopersmith <alan.coopersmith at oracle.com
> <mailto:alan.coopersmith at oracle.com>> wrote:
>
>     Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
>
>     Though I wonder if now you can drop the explicit #define _GNU_SOURCE in
>     linux_sysfs.c now that you moved the config.h up to the same spot, since
>     the AC_USE_SYSTEM_EXTENSIONS in configure.ac <http://configure.ac> should
>     #define _GNU_SOURCE in
>     config.h.
>
> Thanks for the review Alan.
>
> If you don't mind I'll do the removal as a follow up patch.  Will give these
> another 2 days and I'll resend the series with all the comments addressed +
> correct prefix.

Sounds good to me.  (And as for patch 4/4, I have no objections to it, I just
figured someone who actually builds on Linux and knows the difference between
__linux__ and linux #defines should review it instead of me.)

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


More information about the xorg-devel mailing list