Subject: Bug#4012: xbase: `window-managers' was un-correctly generated

From: "H. Manas" <[email-masked]>
Date: Sat, 3 Aug 96 12:08 EDT

Package: xbase
Version: 3.1.2-9

I believe that the 3 first extra explanatory lines in file `xbase.postinst'
of this package should be removed as they trigger unexpected result in some
cases (can go into an infinite loop as Xsession will call itself if you
start X from /etc/X11 for instance).

I'm including the offending portions of the corresponding files here after.

Thanks for the efforts of all Debian's developers and keep the good work
guys.

========================== /etc/X11/Xsession =====================
  if [ -e /etc/X11/window-managers ]
  then
    for i in cat `/etc/X11/window-managers` do
      if [ -x $i ]
      then
        exec $i
      fi
    done
  fi
==================================================================
========================== xbase.postinst ========================
cat >/etc/X11/window-managers <<EOF
# This file contains a list of available window managers. The default
# Xsession file will start the first window manager that it can
^^^^^^^^^^
# in this list.
/usr/X11R6/bin/fvwm
/usr/X11R6/bin/twm
EOF
==================================================================

---

From: "James A. Robinson" <[email-masked]>
Date: Sat, 03 Aug 1996 18:14:50 -0400

> cases (can go into an infinite loop as Xsession will call itself if you

Perhaps the Xsession should be
	
	for i in `grep -v "^#" '/etc/X11/window-managers'` do
or
	for i in `sed -e /^#/d '/etc/X11/window-managers'` do

> ========================== /etc/X11/Xsession =====================
[...]
>     for i in cat `/etc/X11/window-managers` do
[...]