Package: m17n-env
Version: 0.0.3-4
Followup-For: Bug #295332

Sorry i should have read your comments slower:
# This uses the function run_parts defined in /etc/X11/Xsession.
# Not
# to be confused with the Debian utility run-parts, which can
# only
# run files, not list them

/etc/X11/Xsession does not define run_parts any more. It now uses
run-parts --file directory directly.
>From the manual, run-parts --file list all files, not only
executables, so that seems ok.

I have attached my version of the xsession script. It change to
run-parts --list and add a safety when executing
$HOME/.xsession.d/ executable. This is mostly for those that
don't like coffee and thus make a lot of dumb mistakes (as
forgetting to set the shebang in my bash script) ...
It find it great as it tells which is the broken script.
NB: i made this a fatal error, errormsg instead of message, as
without the || it would kill the server anyway. But it may be a
better idea to log the error state and message, process all the
scripts , show all the errors and ask if we should stop the
server. I can make it if you think it is not overkill.

Maybe a similar safety could be added to sourcing , but i really
do not see in which case this could fails (nfs home dir which loose
connection between run-parts and sourcing ... or a solar magnetic
storm of a microsecond who knows )


Regards
Alban


-- System Information:
Debian Release: 3.1
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: i386 (i686)
Kernel: Linux 2.6.11-rc4
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED] (charmap=UTF-8)

Versions of packages m17n-env depends on:
ii  aterm [x-terminal- 0.4.2-10              Afterstep XVT - a VT102 emulator f
ii  ed                 0.2-20                The classic unix line editor
ii  eterm [x-terminal- 0.9.2-8               Enlightened Terminal Emulator
ii  gdm                2.6.0.6-1             GNOME Display Manager
ii  gnome-terminal [x- 2.8.2-1               The GNOME 2 terminal emulator appl
ii  kdm                4:3.3.2-1             KDE Display Manager
ii  konsole [x-termina 4:3.3.2-1             KDE X terminal emulator
ii  kterm [x-terminal- 6.2.0-43              Multi-lingual terminal emulator fo
ii  locales            2.3.2.ds1-20          GNU C Library: National Language (
ii  menu               2.1.21                provides update-menus functions fo
ii  mlterm [x-terminal 2.8.0.cvs20040403-2   MultiLingual TERMinal
ii  multi-gnome-termin 1.6.2-10              Enhanced the GNOME Terminal
ii  rxvt [x-terminal-e 1:2.6.4-6.2           VT102 terminal emulator for the X 
ii  rxvt-unicode [x-te 4.9-1                 RXVT-like terminal emulator with U
ii  terminal [x-termin 0.9.4+cvs20041218-0.1 a Terminal Emulator for GNUstep
ii  xterm [x-terminal- 4.3.0.dfsg.1-10       X terminal emulator

-- no debconf information
# /etc/X11/Xsession.d/80xfree86-common_user-xsession
#
# source ~/.xsession.d/* to set up key user customized X startup
#
# This uses the function run_parts defined in /etc/X11/Xsession. Not
# to be confused with the Debian utility run-parts, which can only
# run files, not list them.
#
# GPL 2.0 (c) 2004 Osamu Aoki per suggestion of Jan Willem Stumpel


if [ -d $HOME/.xsession.d ]; then
  UXSESSIONS=$(run-parts --list $HOME/.xsession.d)
  if [ -n "$UXSESSIONS" ]; then
    for UX in $UXSESSIONS; do
      if [ -x "$UX" ]; then
        "$UX" || errormsg "Sorry, execution of $UX failed. Please check it to 
be a valide executable." 
      else
        . "$UX"
      fi
    done
  fi
fi

Reply via email to