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 <[EMAIL PROTECTED]>  Tue, 27 Sep 2005 22:16:30 -0400
+ -- David Nusinow <[EMAIL PROTECTED]>  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"
+    [EMAIL PROTECTED]@
   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
+        [EMAIL PROTECTED]@
+      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


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to