[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#412069: patch for beryl support



On Mon, Feb 26, 2007 at 11:44:36AM +0100, Robert Millan [ackstorm] wrote:
> On Fri, Feb 23, 2007 at 10:25:56PM +0100, Robert Millan wrote:
> > 
> > Oh, not really.  Screw nVidia.  Beryl is only for faithful followers of the
> > one true Intel card brand.  :-)
> > 
> > So which lines are we going to add, if any?  I can test which lines can
> > Intel do without, but not till monday.
> 
> My intel card can do without either XAANoOffscreenPixmaps or AddARGBGLXVisuals.
> No difference can be found on first sight (and I tested most basic stuff: cube,
> skydome, etc).
> 
> nVidia doesn't need AddARGBGLXVisuals, but it needs XAANoOffscreenPixmaps.
> 
> I don't know about ATI.  But ATI seems to be so badly screwed wrt Beryl that
> we shouldn't care much.

Given all this, can we at least enable the Composite part by default to make
Intel cards work?

Patch attached.

-- 
Robert Millan

ACK STORM, S.L.  -  http://www.ackstorm.es/
diff -ur xorg-7.1.0.old/debian/local/dexconf xorg-7.1.0/debian/local/dexconf
--- xorg-7.1.0.old/debian/local/dexconf	2007-02-13 11:02:09.000000000 +0100
+++ xorg-7.1.0/debian/local/dexconf	2007-02-23 12:07:45.000000000 +0100
@@ -428,6 +433,15 @@
 EndSection
 SECTION
 
+### Extensions
+exec 4>"$DEXCONFTMPDIR/Extensions"
+cat >&4 <<SECTION
+Section "Extensions"
+	# needed for beryl
+	Option		"Composite" "Enable"
+EndSection
+SECTION
+
 # Close file descriptor 4 before we delete temporary files
 exec 4<&-
 
@@ -443,7 +457,7 @@
 
 SPACER=
 for SECTION in Header Files Module InputDeviceKeyboard InputDeviceMouse \
-               Device Monitor Screen ServerLayout DRI; do
+               Device Monitor Screen ServerLayout DRI Extensions; do
   if [ -e "$DEXCONFTMPDIR/$SECTION" ]; then
     eval $SPACER
     cat "$DEXCONFTMPDIR/$SECTION" >>"$OUTFILE"

Reply to: