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

xorg: Changes to 'debian-unstable'



 debian/changelog                |   12 ++++
 debian/xserver-xorg.postinst.in |  100 ++--------------------------------------
 2 files changed, 18 insertions(+), 94 deletions(-)

New commits:
commit 3332df9f724bc50b9a937543df55865d8775d382
Author: David Nusinow <dnusinow@debian.org>
Date:   Sun Nov 4 12:04:13 2007 -0500

    Add closer

diff --git a/debian/changelog b/debian/changelog
index a43d2a3..d2b77ca 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,7 +1,8 @@
 xorg (1:7.3+4) unstable; urgency=low
 
   * Don't try and reset monitor identifier question, it doesn't exist now
-  * Don't bother at all with default-x-server question. Closes: #448863
+  * Don't bother at all with default-x-server question. 
+    Closes: #448863, #374578
     + Remove it from the xserver-xorg postinst. Now instead of a complicated
       md5sum, only create the symlink if nothing exists there.
     + Make sure we delete the old md5sum and xserver roster files for this

commit b422f8e10919fab1d836c6d127a9195fffe929eb
Author: David Nusinow <dnusinow@debian.org>
Date:   Sun Nov 4 11:56:00 2007 -0500

    * Don't bother at all with default-x-server question. Closes: #448863
      + Remove it from the xserver-xorg postinst. Now instead of a complicated
        md5sum, only create the symlink if nothing exists there.
      + Make sure we delete the old md5sum and xserver roster files for this
        stuff. We don't need it now.

diff --git a/debian/changelog b/debian/changelog
index a21b564..a43d2a3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,6 +1,11 @@
 xorg (1:7.3+4) unstable; urgency=low
 
   * Don't try and reset monitor identifier question, it doesn't exist now
+  * Don't bother at all with default-x-server question. Closes: #448863
+    + Remove it from the xserver-xorg postinst. Now instead of a complicated
+      md5sum, only create the symlink if nothing exists there.
+    + Make sure we delete the old md5sum and xserver roster files for this
+      stuff. We don't need it now.
 
  -- David Nusinow <dnusinow@debian.org>  Sun, 04 Nov 2007 11:41:04 -0500
 
diff --git a/debian/xserver-xorg.postinst.in b/debian/xserver-xorg.postinst.in
index 805057b..de79866 100644
--- a/debian/xserver-xorg.postinst.in
+++ b/debian/xserver-xorg.postinst.in
@@ -666,38 +666,6 @@ fi
 
 debug_echo "Configuring $THIS_PACKAGE."
 
-# default X server
-
-# if the X server symlink file already exists and points to an executable X
-# server, it's not as important to ask questions related to it (these questions
-# "have a reasonable default")
-PRIORITY_CEILING=
-if [ -e "$SERVER_SYMLINK" ]; then
-  if [ -x "$(readlink "$SERVER_SYMLINK")" ]; then
-    debug_echo "X server symlink exists and points to executable X server;" \
-               "capping X server question priority at medium."
-    PRIORITY_CEILING=medium
-  fi
-fi
-
-# priority of shared/default-x-server
-PRIORITY=high
-
-db_metaget shared/default-x-server owners
-OWNERS="$RET"
-db_metaget shared/default-x-server choices
-CHOICES="$RET"
-
-if [ "$OWNERS" != "$CHOICES" ]; then
-  debug_echo "\$OWNERS does not equal \$CHOICES: \"$OWNERS\" != \"$CHOICES\""
-  db_subst shared/default-x-server choices $OWNERS
-  db_fset shared/default-x-server seen false
-fi
-
-if ! expr "$OWNERS" : ".*,.*" > /dev/null 2>&1; then
-  debug_echo "\$OWNERS has only one value; shared/default-x-server will not be asked"
-fi
-
 # collect information about installed video card(s), if possible
 case "$ARCH" in
   sparc)
@@ -1692,67 +1660,8 @@ done
 
 # only mess with the server symlink file if it is a symbolic link or does
 # not exist.  otherwise, assume that's the way the user wants it.
-if [ -L "$SERVER_SYMLINK" ] || ! [ -e "$SERVER_SYMLINK" ]; then
-  # migrate from old symlinks.
-  if [ "$(readlink /etc/X11/X)" = "/usr/X11R6/bin/Xorg" ] || \
-     [ "$(readlink /etc/X11/X)" = "/usr/bin/X11/Xorg" ] || \
-     [ "$(readlink /etc/X11/X)" = "/bin/true" ]; then
-    rm /etc/X11/X
-    ln -s $THIS_SERVER /etc/X11/X
-  fi
-
-  # apparently on some broken upgrades the SERVER_SYMLINK_CHECKSUM has been 
-  # lost. If it does not exists, make sure to recreate it.
-
-  if [ ! -e "$SERVER_SYMLINK_CHECKSUM" ] && [ -L "$SERVER_SYMLINK" ]; then
-    readlink "$SERVER_SYMLINK" | md5sum > "$SERVER_SYMLINK_CHECKSUM"
-  fi
-
-  # compare the current and stored checksums; if they do not match, assume
-  # that's the way the user wants it.
-  # why, why, why, why, why, why, why are we md5suming this? -daniels
-  if [ "$(readlink "$SERVER_SYMLINK" | md5sum)" = \
-       "$(cat "$SERVER_SYMLINK_CHECKSUM")" ] || \
-     [ "$(echo "/usr/X11R6/bin/Xorg" | md5sum)" = \
-       "$(cat "$SERVER_SYMLINK_CHECKSUM")" ] || \
-     [ "$(echo "/usr/bin/X11/Xorg" | md5sum)" = \
-       "$(cat "$SERVER_SYMLINK_CHECKSUM")" ] || \
-     [ "$(echo "/usr/bin/X11/XFree86" | md5sum)" = \
-       "$(cat "$SERVER_SYMLINK_CHECKSUM")" ] || \
-     [ "$(echo "/bin/true" | md5sum)" = \
-       "$(cat "$SERVER_SYMLINK_CHECKSUM")" ] || \
-     ! [ -e "$SERVER_SYMLINK" ]; then
-    # finally, only update the symlink if this package contains the
-    # selected default X server.
-    db_get shared/default-x-server
-    if [ -n "$RET" ]; then
-      SELECTED_PACKAGE="$RET"
-      if [ "$SELECTED_PACKAGE" = "$THIS_PACKAGE" ]; then
-        # prepare a new version of the config file; this is a symlink so we
-        # can't use the tempfile command for it (we'd have to subsequently
-        # use ln -sf, which is subject to race condition attacks)
-        NEW_SERVER_SYMLINK="$SERVER_SYMLINK.dpkg-new"
-        ln -sf "$THIS_SERVER" "$NEW_SERVER_SYMLINK"
-        if ! cmp -s "$SERVER_SYMLINK" "$NEW_SERVER_SYMLINK"; then
-          if [ "$(readlink "$SERVER_SYMLINK")" \
-               != "$(readlink "$NEW_SERVER_SYMLINK")" ]; then
-            mv "$NEW_SERVER_SYMLINK" "$SERVER_SYMLINK"
-            readlink "$SERVER_SYMLINK" | md5sum > \
-                     "$SERVER_SYMLINK_CHECKSUM"
-          fi
-        fi
-        rm -f "$NEW_SERVER_SYMLINK"
-      fi
-    else
-      warn "not updating $SERVER_SYMLINK; no default X server" \
-           "configured; run \"dpkg-reconfigure $THIS_PACKAGE\" to" \
-           "correct this"
-    fi
-  else
-    warn "not updating $SERVER_SYMLINK; file has been customized"
-  fi
-else
-  warn "not updating $SERVER_SYMLINK; file exists but is not a symlink"
+if ! [ -e "$SERVER_SYMLINK" ]; then
+  ln -s "$THIS_SERVER" "$SERVER_SYMLINK"
 fi
 
 # Don't touch the config on upgrades except to deal with known issues with old
@@ -1840,6 +1749,9 @@ else
   debug_echo "not updating $XORGCONFIG; we're upgrading"
 fi
 
+# Clean up our old crap
+rm -f "$SERVER_SYMLINK_CHECKSUM" "$SERVER_SYMLINK_ROSTER"
+
 exit 0
 
 # vim:set ai et sts=2 sw=2 tw=0:

commit 512af73357dcb43d52c09d1ea014567f0a5bbf75
Author: David Nusinow <dnusinow@debian.org>
Date:   Sun Nov 4 11:41:24 2007 -0500

    * Don't try and reset monitor identifier question, it doesn't exist now

diff --git a/debian/changelog b/debian/changelog
index af7d229..a21b564 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+xorg (1:7.3+4) unstable; urgency=low
+
+  * Don't try and reset monitor identifier question, it doesn't exist now
+
+ -- David Nusinow <dnusinow@debian.org>  Sun, 04 Nov 2007 11:41:04 -0500
+
 xorg (1:7.3+3) unstable; urgency=low
 
   [ David Nusinow ]
diff --git a/debian/xserver-xorg.postinst.in b/debian/xserver-xorg.postinst.in
index 529bfef..805057b 100644
--- a/debian/xserver-xorg.postinst.in
+++ b/debian/xserver-xorg.postinst.in
@@ -1203,7 +1203,7 @@ xresprobeint() {
     VERT_REFRESH=
 
     # clear out all values
-    for i in identifier horiz-sync vert-refresh use_sync_ranges \
+    for i in horiz-sync vert-refresh use_sync_ranges \
              selection-method screen-size; do
       db_reset xserver-xorg/config/monitor/$i
     done



Reply to: