rpms/xorg-x11-server/OLPC-3 swrast-dont-abort.patch, 1.1, 1.2 xorg-x11-server.spec, 1.339, 1.340
Daniel Drake (dsd)
fedora-extras-commits at redhat.com
Fri Jul 25 15:57:31 UTC 2008
Author: dsd
Update of /cvs/pkgs/rpms/xorg-x11-server/OLPC-3
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv2076
Modified Files:
swrast-dont-abort.patch xorg-x11-server.spec
Log Message:
update swrast patch to upstream version, fixes errno bug
swrast-dont-abort.patch:
Index: swrast-dont-abort.patch
===================================================================
RCS file: /cvs/pkgs/rpms/xorg-x11-server/OLPC-3/swrast-dont-abort.patch,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- swrast-dont-abort.patch 24 Jul 2008 19:06:12 -0000 1.1
+++ swrast-dont-abort.patch 25 Jul 2008 15:57:01 -0000 1.2
@@ -1,30 +1,32 @@
-From 6bf46df463d19456761ae670f60e0b153e285bd9 Mon Sep 17 00:00:00 2001
+From eff25430b4a391409e39337962ff7697165d23c7 Mon Sep 17 00:00:00 2001
From: Daniel Drake <dsd at laptop.org>
-Date: Thu, 24 Jul 2008 14:42:22 -0400
+Date: Thu, 24 Jul 2008 21:06:34 -0500
Subject: [PATCH] Don't abort if swrast library is not present
+GLX is enabled by default, but the current swrast behaviour causes X
+to abort with fatal error if the swrast dri library dlopen fails.
+
+Handle the case where the swrast library is not present, and do not
+register the GLX extension unless at least one screen has a usable
+GL provider.
---
- glx/glxdriswrast.c | 6 ++++++
+ glx/glxdriswrast.c | 2 +-
glx/glxext.c | 37 ++++++++++++++++++++++++-------------
- 2 files changed, 30 insertions(+), 13 deletions(-)
+ 2 files changed, 25 insertions(+), 14 deletions(-)
diff --git a/glx/glxdriswrast.c b/glx/glxdriswrast.c
-index cf4827e..20c39ab 100644
+index 199f5f8..2859704 100644
--- a/glx/glxdriswrast.c
+++ b/glx/glxdriswrast.c
-@@ -468,6 +468,12 @@ __glXDRIscreenProbe(ScreenPtr pScreen)
+@@ -515,7 +515,7 @@ __glXDRIscreenProbe(ScreenPtr pScreen)
- screen->driver = dlopen(filename, RTLD_LAZY | RTLD_LOCAL);
- if (screen->driver == NULL) {
-+ if (errno == ENOENT) {
-+ xfree(screen);
-+ LogMessage(X_INFO, "AIGLX: swrast library %s not found\n",
-+ filename);
-+ return NULL;
-+ }
- LogMessage(X_ERROR, "AIGLX error: dlopen of %s failed (%s)\n",
- filename, dlerror());
- goto handle_error;
+ xfree(screen);
+
+- FatalError("GLX: could not load software renderer\n");
++ LogMessage(X_ERROR, "GLX: could not load software renderer\n");
+
+ return NULL;
+ }
diff --git a/glx/glxext.c b/glx/glxext.c
index 13c65da..6ba404f 100644
--- a/glx/glxext.c
Index: xorg-x11-server.spec
===================================================================
RCS file: /cvs/pkgs/rpms/xorg-x11-server/OLPC-3/xorg-x11-server.spec,v
retrieving revision 1.339
retrieving revision 1.340
diff -u -r1.339 -r1.340
--- xorg-x11-server.spec 24 Jul 2008 19:06:12 -0000 1.339
+++ xorg-x11-server.spec 25 Jul 2008 15:57:01 -0000 1.340
@@ -19,7 +19,7 @@
Summary: X.Org X11 X server
Name: xorg-x11-server
Version: 1.4.99.906
-Release: 1%{?dist}
+Release: 2%{?dist}
URL: http://www.x.org
License: MIT
Group: User Interface/X
@@ -462,6 +462,10 @@
%changelog
+* Fri Jul 25 2008 Daniel Drake <dsd at laptop.org> 1.4.99.906-2
+- update no-swrast patch to the one that went upstream, this one had a bug
+ (dlopen isn't documented to populate errno)
+
* Thu Jul 24 2008 Daniel Drake <dsd at laptop.org> 1.4.99.906-1
- resync with F-9 for new xinput ABI
- add patch to allow X server to run when mesa's swrast is not present
More information about the fedora-extras-commits
mailing list