summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog10
-rw-r--r--data/Init.in8
-rwxr-xr-xdata/PostSession.in24
-rwxr-xr-xdata/PreSession.in31
4 files changed, 14 insertions, 59 deletions
diff --git a/ChangeLog b/ChangeLog
index 4c09951a..e7b40b7d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2008-12-08 Ray Strode <rstrode@redhat.com>
+
+ * data/{PreSession,PostSession}.in:
+ gut functions, since they were redundant
+ with what we do in code now (bug 563550)
+
+ * data/Init.in:
+ apply patch from Pacho Ramos to quote
+ shell conditionals better (bug 563550).
+
2008-12-03 William Jon McCann <jmccann@redhat.com>
* configure.ac: post bump
diff --git a/data/Init.in b/data/Init.in
index 16ba078b..091f5351 100644
--- a/data/Init.in
+++ b/data/Init.in
@@ -32,8 +32,8 @@ fi
sysmodmap=/etc/X11/Xmodmap
XMODMAP=`gdmwhich xmodmap`
-if [ x$XMODMAP != x ] ; then
- if [ x$GDM_PARENT_DISPLAY = x ]; then
+if [ "x$XMODMAP" != "x" ] ; then
+ if [ "x$GDM_PARENT_DISPLAY" = "x" ]; then
if [ -f $sysmodmap ]; then
$XMODMAP $sysmodmap
fi
@@ -47,7 +47,7 @@ if [ x$XMODMAP != x ] ; then
UNAME=`gdmwhich uname`
PROCESSOR=`$UNAME -p`
- if [ x$PROCESSOR = xsparc ]; then
+ if [ "x$PROCESSOR" = "xsparc" ]; then
if $XMODMAP | /usr/bin/grep mod4 | /usr/bin/grep Alt > /dev/null 2>/dev/null
then
$XMODMAP -e "clear Mod1" \
@@ -61,7 +61,7 @@ if [ x$XMODMAP != x ] ; then
fi
SETXKBMAP=`gdmwhich setxkbmap`
-if [ x$SETXKBMAP != x ] ; then
+if [ "x$SETXKBMAP" != "x" ] ; then
# FIXME: is this all right? Is this completely on crack?
# What this does is move the xkb configuration from the GDM_PARENT_DISPLAY
# FIXME: This should be done in code. Or there must be an easier way ...
diff --git a/data/PostSession.in b/data/PostSession.in
index 7c24e44c..c52d3c26 100755
--- a/data/PostSession.in
+++ b/data/PostSession.in
@@ -1,27 +1,3 @@
#!/bin/sh
-PATH="@X_PATH@:$PATH:/bin:/usr/bin"
-OLD_IFS=$IFS
-
-gdmwhich () {
- COMMAND="$1"
- OUTPUT=
- IFS=:
- for dir in $PATH
- do
- if test -x "$dir/$COMMAND" ; then
- if test "x$OUTPUT" = "x" ; then
- OUTPUT="$dir/$COMMAND"
- fi
- fi
- done
- IFS=$OLD_IFS
- echo "$OUTPUT"
-}
-
-SESSREG=`gdmwhich sessreg`
-if [ "x$SESSREG" != "x" ] ; then
- "$SESSREG" -d -w /var/log/wtmp -u /var/run/utmp -x "$X_SERVERS" -h "$REMOTE_HOST" -l "$DISPLAY" "$USER"
-fi
-
exit 0
diff --git a/data/PreSession.in b/data/PreSession.in
index 8e48f793..6f31d3d9 100755
--- a/data/PreSession.in
+++ b/data/PreSession.in
@@ -7,34 +7,3 @@
# Note that output goes into the .xsession-errors file for easy debugging
#
PATH="@X_PATH@:$PATH:/bin:/usr/bin"
-OLD_IFS=$IFS
-
-gdmwhich () {
- COMMAND="$1"
- OUTPUT=
- IFS=:
- for dir in $PATH
- do
- if test -x "$dir/$COMMAND" ; then
- if test "x$OUTPUT" = "x" ; then
- OUTPUT="$dir/$COMMAND"
- fi
- fi
- done
- IFS=$OLD_IFS
- echo "$OUTPUT"
-}
-
-SESSREG=`gdmwhich sessreg`
-if [ "x$SESSREG" != "x" ] ; then
- # some output for easy debugging
- echo "$0: Registering your session with wtmp and utmp"
- echo "$0: running: $SESSREG -a -w /var/log/wtmp -u /var/run/utmp -x \"$X_SERVERS\" -h \"$REMOTE_HOST\" -l \"$DISPLAY\" \"$USER\""
-
- exec "$SESSREG" -a -w /var/log/wtmp -u /var/run/utmp -x "$X_SERVERS" -h "$REMOTE_HOST" -l "$DISPLAY" "$USER"
- # this is not reached
-fi
-
-# some output for easy debugging
-echo "$0: could not find the sessreg utility, cannot update wtmp and utmp"
-exit 0