Author: branden Date: 2003-10-27 12:15:25 -0500 (Mon, 27 Oct 2003) New Revision: 720
Modified: trunk/debian/changelog trunk/debian/xdm.config.in Log: Fix xdm's config script to not choke on multiword garbage in /etc/X11/default-display-manager, by quoting the argument to basename. Use grep with head to extract the first non-whitespace, non-comment line from the file. If the result does not appear to be a command ("which" fails), observe() it. (Closes: #217605) - debian/xdm.config.in Modified: trunk/debian/changelog =================================================================== --- trunk/debian/changelog 2003-10-27 08:49:38 UTC (rev 719) +++ trunk/debian/changelog 2003-10-27 17:15:25 UTC (rev 720) @@ -255,8 +255,15 @@ - debian/xfonts-scalable.postinst.in - debian/xfonts-scalable.postrm.in - -- Branden Robinson <[EMAIL PROTECTED]> Mon, 27 Oct 2003 03:48:42 -0500 + * Fix xdm's config script to not choke on multiword garbage in + /etc/X11/default-display-manager, by quoting the argument to basename. + Use grep with head to extract the first non-whitespace, non-comment line + from the file. If the result does not appear to be a command ("which" + fails), observe() it. (Closes: #217605) + - debian/xdm.config.in + -- Branden Robinson <[EMAIL PROTECTED]> Mon, 27 Oct 2003 11:51:41 -0500 + xfree86 (4.2.1-12.1) unstable; urgency=low * Fix typo in xlibs preinst. Closes: #213774, #213776 Modified: trunk/debian/xdm.config.in =================================================================== --- trunk/debian/xdm.config.in 2003-10-27 08:49:38 UTC (rev 719) +++ trunk/debian/xdm.config.in 2003-10-27 17:15:25 UTC (rev 720) @@ -33,8 +33,15 @@ # debconf is not a registry; use the current contents of the default display # manager file to pre-answer the question if possible if [ -e "$DEFAULT_DISPLAY_MANAGER_FILE" ]; then - CURRENT_DEFAULT=$(basename $(head -n 1 "$DEFAULT_DISPLAY_MANAGER_FILE")) + CURRENT_DEFAULT=$(basename "$(grep -v '^[[:space:]]*#' \ + "$DEFAULT_DISPLAY_MANAGER_FILE" | + head -n -1)") if [ -n "$CURRENT_DEFAULT" ]; then + if ! which "$CURRENT_DEFAULT" > /dev/null 2>&1; then + observe "default display manager \"$CURRENT_DEFAULT\" specified in" \ + "$DEFAULT_DISPLAY_MANAGER_FILE does not exist or is not" \ + "executable" + fi safe_debconf db_set shared/default-x-display-manager "$CURRENT_DEFAULT" fi else -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]