[PATCH libpciaccess 4/5] Check for __linux__ instead of linux during compilation

Emil Velikov emil.l.velikov at gmail.com
Tue Feb 10 06:55:22 PST 2015


From: Eero Tamminen <eero.t.tamminen at intel.com>

"__linux__" is the POSIX define for checking for Linux OS, "linux" is
deprecated and apparently not supported by Android.

Besides correcting the define, patch adds error for the case OS isn't
recognized.

Signed-off-by: Eero Tamminen <eero.t.tamminen at intel.com>

v2: Drop double negative from commit msg. Spotted by Mateusz Jończyk.
[Emil Velikov: Remove irrelevant Android details from commit msg.]
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
Reviewed-by: Jasper St. Pierre <jstpierre at mecheye.net>
---
 src/common_init.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/common_init.c b/src/common_init.c
index a127a8b..b1c0c3e 100644
--- a/src/common_init.c
+++ b/src/common_init.c
@@ -52,7 +52,7 @@ pci_system_init( void )
 {
     int err = ENOSYS;
 
-#ifdef linux
+#ifdef __linux__
     err = pci_system_linux_sysfs_create();
 #elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__)
     err = pci_system_freebsd_create();
@@ -64,6 +64,8 @@ pci_system_init( void )
     err = pci_system_solx_devfs_create();
 #elif defined(__GNU__) || defined(__CYGWIN__)
     err = pci_system_x86_create();
+#else
+# error "Unsupported OS"
 #endif
 
     return err;
-- 
2.2.2



More information about the xorg-devel mailing list