<div dir="ltr"><div>This makes sense.<br><br></div>Reviewed-by: Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net">jstpierre@mecheye.net</a>><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 9, 2015 at 1:56 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Would anyone have some input - apart from the missing libpciaccess<br>
prefix of course ?<br>
Fwiw src/common_interface.c already checks for __linux__, so this<br>
patch makes things more consistent as an added bonus :-P<br>
<br>
Cheers,<br>
Emil<br>
<div class="HOEnZb"><div class="h5"><br>
On 6 February 2015 at 17:29, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br>
> From: Eero Tamminen <<a href="mailto:eero.t.tamminen@intel.com">eero.t.tamminen@intel.com</a>><br>
><br>
> "__linux__" is the POSIX define for checking for Linux OS, "linux" is<br>
> deprecated and apparently not supported by Android.<br>
><br>
> Besides correcting the define, patch adds error for the case OS isn't<br>
> unrecognized.<br>
><br>
> Signed-off-by: Eero Tamminen <<a href="mailto:eero.t.tamminen@intel.com">eero.t.tamminen@intel.com</a>><br>
> [Emil Velikov: Remove irrelevant Android details from commit msg.]<br>
> Signed-off-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
> ---<br>
>  src/common_init.c | 4 +++-<br>
>  1 file changed, 3 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/common_init.c b/src/common_init.c<br>
> index a127a8b..b1c0c3e 100644<br>
> --- a/src/common_init.c<br>
> +++ b/src/common_init.c<br>
> @@ -52,7 +52,7 @@ pci_system_init( void )<br>
>  {<br>
>      int err = ENOSYS;<br>
><br>
> -#ifdef linux<br>
> +#ifdef __linux__<br>
>      err = pci_system_linux_sysfs_create();<br>
>  #elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__)<br>
>      err = pci_system_freebsd_create();<br>
> @@ -64,6 +64,8 @@ pci_system_init( void )<br>
>      err = pci_system_solx_devfs_create();<br>
>  #elif defined(__GNU__) || defined(__CYGWIN__)<br>
>      err = pci_system_x86_create();<br>
> +#else<br>
> +# error "Unsupported OS"<br>
>  #endif<br>
><br>
>      return err;<br>
> --<br>
> 2.2.2<br>
><br>
_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a></div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">  Jasper<br></div>
</div>