<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
On Thu, 2010-07-01 at 19:52 -0700, Alan Coopersmith wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
Both #include Linux specific headers such as &lt;linux/types.h&gt; so can't
be built on non-Linux platforms.

Signed-off-by: Alan Coopersmith &lt;<A HREF="mailto:alan.coopersmith@oracle.com">alan.coopersmith@oracle.com</A>&gt;
---
 build.sh |   18 ++++++++++--------
 1 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/build.sh b/build.sh
index 5eb2cd3..d5884d8 100755
--- a/build.sh
+++ b/build.sh
@@ -509,6 +509,7 @@ build_driver_video() {
             build driver xf86-video-sisusb
             build driver xf86-video-sunffb
             build driver xf86-video-v4l
+            build driver xf86-video-xgixp
             ;;
         *)
             ;;
@@ -531,6 +532,15 @@ build_driver_video() {
             ;;
     esac
 
+    # Some drivers are only buildable on some architectures of some OS's
+    case &quot;$HOST_CPU&quot;-&quot;$HOST_OS&quot; in
+        i*86*-*Linux*)
+            build driver xf86-video-geode
+            ;;
+        *)
+            ;;
+    esac
+
     build driver xf86-video-apm
     build driver xf86-video-ark
     build driver xf86-video-ast
@@ -539,13 +549,6 @@ build_driver_video() {
     build driver xf86-video-cirrus
     build driver xf86-video-dummy
     build driver xf86-video-fbdev
-    case $HOST_CPU in
-        i*86*)
-            build driver xf86-video-geode
-            ;;
-        *)
-            ;;
-    esac
 #    build driver xf86-video-glide
     build driver xf86-video-glint
     build driver xf86-video-i128
@@ -571,7 +574,6 @@ build_driver_video() {
     build driver xf86-video-vmware
     build driver xf86-video-voodoo
     build driver xf86-video-xgi
-    build driver xf86-video-xgixp
 }
 
 # The server must be built before the drivers
</PRE>
</BLOCKQUOTE>
<BR>
Reviewed-by: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;<BR>
<BR>
</BODY>
</HTML>