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

X Strike Force X.Org X11 SVN commit: r633 - trunk/debian



Author: dnusinow
Date: 2005-09-27 21:35:38 -0500 (Tue, 27 Sep 2005)
New Revision: 633

Modified:
   trunk/debian/changelog
   trunk/debian/rules
   trunk/debian/xserver-xorg.config.in
   trunk/debian/xserver-xorg.postinst.in
Log:
* Don't choke when the user doesn't select a video mode. This fix also
  involves a patch to substitute the default resolutions list in both
  the xserver-xorg.preinst and .postinst files using part of
  debian/rules. Thanks Robert Millan. (closes: #322902, #330304)
  + This commit differs from the previous in that I applied a patch from
    Robert for a more complete fix.


Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2005-09-28 02:18:40 UTC (rev 632)
+++ trunk/debian/changelog	2005-09-28 02:35:38 UTC (rev 633)
@@ -38,7 +38,10 @@
   * Update general/055_lnx_evdev_keyboard.diff to properly use the "less
     greater bar"-key (EV_KEY_102ND) on a pc102 keyboards. Thanks Pär
     Andersson. (closes: #322253)
-  * Don't choke when the user doesn't select a video mode. (closes: #322902)
+  * Don't choke when the user doesn't select a video mode. This fix also
+    involves a patch to substitute the default resolutions list in both
+    the xserver-xorg.preinst and .postinst files using part of
+    debian/rules. Thanks Robert Millan. (closes: #322902, #330304)
 
   [ Branden Robinson ]
   * Change xserver-xorg and xserver-xorg-dbg's recommendation of "discover1"
@@ -52,7 +55,7 @@
   * Updated Czech translation (cs.po). Thanks, Miroslav Kure (closes:
     #327506).
 
- -- David Nusinow <dnusinow@debian.org>  Tue, 27 Sep 2005 22:16:30 -0400
+ -- David Nusinow <dnusinow@debian.org>  Tue, 27 Sep 2005 22:33:29 -0400
 
 xorg-x11 (6.8.2.dfsg.1-7) unstable; urgency=high
 

Modified: trunk/debian/rules
===================================================================
--- trunk/debian/rules	2005-09-28 02:18:40 UTC (rev 632)
+++ trunk/debian/rules	2005-09-28 02:35:38 UTC (rev 633)
@@ -76,6 +76,8 @@
 SERVER_WRAPPER_OPT=-O2
 endif
 
+DEFAULT_DCRESOLUTIONS="1920x1440, 1920x1200, 1856x1392, 1792x1344, 1680x1050, 1600x1200, 1440x900, 1400x1050, 1280x1024, 1280x960, 1280x854, 1280x800, 1280x768, 1200x800, 1152x864, 1152x768, 1024x768, 800x600, 640x480"
+
 # Set up parameters for the Debian build environment.
 
 # Determine our architecture.
@@ -288,6 +290,7 @@
 	      | sed -e '/^#INCLUDE_SHELL_LIB#$$/d' >>$$MAINTSCRIPT.tmp; \
 	    sed -e 's/@SOURCE_VERSION@/$(SOURCE_VERSION)/' \
 	        -e 's/@OFFICIAL_BUILD@/$(OFFICIAL_BUILD)/' \
+	        -e 's/@DEFAULT_DCRESOLUTIONS@/$(DEFAULT_DCRESOLUTIONS)/' \
 	      <$$MAINTSCRIPT.tmp >$$MAINTSCRIPT; \
 	    rm $$MAINTSCRIPT.tmp; \
 	  fi; \

Modified: trunk/debian/xserver-xorg.config.in
===================================================================
--- trunk/debian/xserver-xorg.config.in	2005-09-28 02:18:40 UTC (rev 632)
+++ trunk/debian/xserver-xorg.config.in	2005-09-28 02:35:38 UTC (rev 633)
@@ -486,8 +486,7 @@
   if [ "$RET" != "10 xserver-xfree86/config/display/modes doesn't exist" ]; then
     db_subst xserver-xorg/config/display/modes choices "$RET"
   else
-    # remember to keep this list in sync with postinst.in. Yes, it sucks!
-    db_subst xserver-xorg/config/display/modes choices "1920x1440, 1920x1200, 1856x1392, 1792x1344, 1680x1050, 1600x1200, 1440x900, 1400x1050, 1280x1024, 1280x960, 1280x854, 1280x800, 1280x768, 1200x800, 1152x864, 1152x768, 1024x768, 800x600, 640x480"
+    db_subst xserver-xorg/config/display/modes choices @DEFAULT_DCRESOLUTIONS@
   fi
   # migrate keyboard/rules
   db_get xserver-xorg/config/inputdevice/keyboard/rules

Modified: trunk/debian/xserver-xorg.postinst.in
===================================================================
--- trunk/debian/xserver-xorg.postinst.in	2005-09-28 02:18:40 UTC (rev 632)
+++ trunk/debian/xserver-xorg.postinst.in	2005-09-28 02:35:38 UTC (rev 633)
@@ -394,7 +394,7 @@
     DCRESOLUTIONS="$RET"
     DCRESOLUTIONS=$(echo $DCRESOLUTIONS | sed -e 's/, / /g')
   else
-    DCRESOLUTIONS="1920x1440 1920x1200 1856x1392 1792x1344 1680x1050 1600x1200 1440x900 1400x1050 1280x1024 1280x960 1280x854 1280x800 1280x768 1200x800 1152x864 1152x768 1024x768 800x600 640x480"
+    DCRESOLUTIONS=@DEFAULT_DCRESOLUTIONS@
   fi
   if [ -n "$RESOLUTIONS" ]; then
     DCRESOLUTIONS=$(for i in $DCRESOLUTIONS $RESOLUTIONS; do echo $i; done | sort -unr)
@@ -423,6 +423,9 @@
       # if we are here we did not detect the frequencies, but we have the resolution.
       # get the highest resolution.
       db_get xserver-xorg/config/display/modes
+      if [ -z "$RET" ]; then
+        RET=@DEFAULT_DCRESOLUTIONS@
+      fi
       MAXRES=$(echo $RET | sed -e 's/,//g')
       MAXRES=$(for i in $MAXRES; do echo $i; done | sort -unr | head -n 1)
       # set a sane default for mode-list



Reply to: