Fix check for adding /usr/bin to the GDM_USER_PATH so it doesn't add it

2006-04-13  Brian Cameron  <brian.cameron@sun.com>

        * configure.ac: Fix check for adding /usr/bin to the GDM_USER_PATH
          so it doesn't add it twice.  The previous code didn't really work.

3 files changed, 26 insertions, 24 deletions

diff --git a/ChangeLog b/ChangeLog
index 910b21ba..3fb9c330 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,8 @@
 	* NEWS: Updated NEWS.
 	* gui/gdmchooser.c: Fix references to the BROADCAST key so it
 	  is correctly listed as a bool, not a string.
+	* configure.ac: Fix check for adding /usr/bin to the GDM_USER_PATH
+	  so it doesn't add it twice on Solaris.
 
 2006-04-11  Brian Cameron  <brian.cameron@sun.com>
 
diff --git a/configure.ac b/configure.ac
index 0541b465..6fcb1c2c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1003,16 +1003,16 @@ else
 fi
 
 # Don't add bindir to GDM_USER_PATH if it is already in the PATH.
-# If it is /usr/bin, for example.
-if test "x${bindir}" != 'x${exec_prefix}/bin' -a "x${exec_prefix}" = 'x${prefix}'; then
-    if ${prefix} != / -a ${prefix} != /bin; then
-        GDM_USER_PATH="$GDM_USER_PATH:${bindir}"
-    fi
-elif test "x${bindir}" != 'x${exec_prefix}/bin' -a "x${exec_prefix}" != 'x${prefix}'; then
-    if ${exec_prefix} != / -a ${exec_prefix} != /bin; then
-        GDM_USER_PATH="$GDM_USER_PATH:${bindir}"
-    fi
-elif test "x${bindir}" != 'x/bin'; then
+# If it is /bin or /usr/bin, for example.
+if test ${exec_prefix} != "NONE"; then
+   path_to_check=$exec_prefix
+elif test ${prefix} != "NONE"; then
+   path_to_check=$prefix
+else
+   path_to_check=$ac_default_prefix
+fi
+
+if test "x$path_to_check" != "x/" -a "x$path_to_check" != "x/usr"; then
     GDM_USER_PATH="$GDM_USER_PATH:${bindir}"
 fi
 
@@ -1194,9 +1194,9 @@ fi
 
 dnl <= Solaris Trusted Extensions support =>
 if test "x$found_tsol" = "xyes" ; then
-	echo "Solaris Trusted Extensions support	: YES"
+	echo "Solaris Trusted Extensions support    : YES"
 else
-	echo "Solaris Trusted Extensions support	: NO"
+	echo "Solaris Trusted Extensions support    : NO"
 fi
 
 dnl <= Authentication scheme =>
diff --git a/docs/es/gdm.xml b/docs/es/gdm.xml
index 90b1e73b..d2b28dc3 100644
--- a/docs/es/gdm.xml
+++ b/docs/es/gdm.xml
@@ -279,7 +279,7 @@
 
       <para>El peligro es que alguien que gane privilegios del usuario/grupo GDM puede entonces conectarse a cualquier sesión. Así que no debería bajo ninguna circunstancia, hacer esto a algún usuario/grupo que pueda ser fácil obtener acceso, como el usuario <filename>nobody</filename>. Los usuarios que ganan acceso al usuario «gdm» podrían también modificar las claves Xauth y provocar ataques de denegación de servicio. También si una persona gana la capacidad de ejecutar programas como el usuario «gdm», podría ser posible espiar los procesos GDM en ejecución, incluyendo usuarios y contraseñas a medida que se teclean.</para>
 
-      <para>Distributions and system administrators using GDM are expected to setup the dedicated user properly. It is recommended that this userid be configured to disallow login and to not have a default shell. Distributions and system administrators should set up the filesystem to ensure that the GDM user does not have read or write access to sensitive files.</para>
+      <para>Se espera que las distribuciones y los administradores de sistemas que usen GDM configuren el usuario dedicado de forma apropiada. Se recomienda que este id de usuario se configure para no permitirle iniciar sesión y no tener un intérprete de comandos predeterminado. Las distribuciones y los administradores del sistema deberían configurar el sistema de archivos para asegurarse de que el usuario GDM no tenga permiso de lectura o escritura a los archivos sensibles.</para>
     </sect2>
 
     <sect2 id="xauth">
@@ -344,11 +344,11 @@ gdm: .su.dominio
 
     <para>Esta sección cubrirá la configuración de GDM y el formato de los archivos de configuración de GDM. Puede usar el comando <command>gdmsetup</command> para configurar GDM, pero la aplicación de configuración no le deja configurar cada aspecto de GDM.</para>
 
-    <para>You can either run <command>gdmsetup</command> as root or turn on the GDM configuration feature that allows you to run <command>gdmsetup</command> from the login screen (where it will run as root). If you believe running root owned GUI's is a security risk, then you would want to edit the files by hand.</para>
+    <para>Puede ejecutar o el comando <command>gdmsetup</command> como root o activar la característica de configuración que le permite ejecutar <command>gdmsetup</command> desde la pantalla de entrada (donde se ejecuta como root). Si cree que ejecutar IGU's como root es un riesgo de seguridad, entonces quizá quiera editar estos archivos a mano.</para>
 
-    <para>If you are a distribution and want to set machine defaults, you should edit the <filename>&lt;share&gt;/gdm/defaults.conf</filename> file rather than editing the <filename>&lt;etc&gt;/gdm/custom.conf</filename> file, so the distribution changes are preserved as defaults.</para>
+    <para>Si está preparando una distribución y quiere establecer configuraciones predeterminadas para la máquina, debería editar el archivo <filename>&lt;share&gt;/gdm/defaults.conf</filename> en vez de editar el archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> file, para los cambios de la distribución se preserven como predeterminados.</para>
 
-    <para>If you want to change configuration by hand, edit the <filename>&lt;etc&gt;/gdm/custom.conf</filename> file and make sure the keyname=value pair you want is included in the appropriate section. For example, to change the "Greeter" key in the "daemon" section, make sure the daemon section of the <filename>&lt;etc&gt;/gdm/custom.conf</filename> file has the value like in this example. The modified option does not have to come anywhere in the section. The <command>gdmsetup</command> program manages editing these files for you.</para>
+    <para>Si quiere cambiar la configuración a mano, edite el archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> y asegúrese de que el par clave=valor que quiere está incluido en la sección apropiada. Por ejemplo, para cambiar la clave  "Greeter" en la sección "daemon" asegúrese de que la sección "daemon" del archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> tiene el valor como en el ejemplo. La opción modificada no tiene que venir de en cualquier lugar en la sección. El comando <command>gdmsetup</command> gestiona la edición de estos archivos por usted.</para>
 
 <screen>
 [daemon]
@@ -359,9 +359,9 @@ Greeter=/usr/lib/gdmgreeter
 
     <para>Algunas claves en el archivo de configuración se entregan comentadas mientras que otras están activadas. Esto se hace para que los predeterminados para algunas claves puedan cambiarse fácilmente en el futuro. GDM considera que las líneas que comienzan con el carácter «#» un comentario,  y estas líneas se ignorarán.</para>
 
-    <para>The <filename>&lt;share&gt;/gdm/defaults.conf</filename> file contains the default configuration choices for GDM, and should not be modified by the user. The <filename>&lt;etc&gt;/gdm/custom.conf</filename> file is where users may specify their custom configuration choices. Configuration options specified in the <filename>&lt;etc&gt;/gdm/custom.conf</filename> file override the values in the main <filename>&lt;share&gt;/gdm/defaults.conf</filename> file. Running the <command>gdmsetup</command> command will cause the <filename>&lt;etc&gt;/gdm/custom.conf</filename> to be modified with the user's configuration choices and will cause any running GDM GUI programs to automatically update. Previous to version 2.13.0.4 GDM only supported the <filename>&lt;etc&gt;/gdm/gdm.conf</filename> file, so if using an older version of GDM just edit that file directly.</para>
+    <para>EL archivo  <filename>&lt;share&gt;/gdm/defaults.conf</filename> contiene las opciones de configuración predeterminadas para GDM, y no debería ser modificado por el usuario. El archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> es donde los usuarios pueden especificar sus opciones de configuración personalizadas. Las opciones de configuración especificadas en el archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> toman precedencia sobre los valores en el archivo principal <filename>&lt;share&gt;/gdm/defaults.conf</filename>. Ejecutar el comando <command>gdmsetup</command> hará que el archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> se modifique con las opciones de configuración del usuario y hará que cualquier programa del IGU de GDM se actualice automáticamente. Anteriormente a la versión 2.13.0.4 GDM sólo soportaba el archivo <filename>&lt;etc&gt;/gdm/gdm.conf</filename>, así que si está usando una versión más antigua de GDM tan sólo edite ese archivo directamente.</para>
 
-    <para>The location of the configuration files may be controlled via the <command>--with-defaults-conf</command> and <command>--with-custom-conf</command> configuration options. The GDM daemon --config option may also be used to specify the configuration file location. The GDM daemon must be restarted to change the configuration file being used.</para>
+    <para>La ubicación de los archivos de configuración puede controlarse por medio de las opciones de configuración <command>--with-defaults-conf</command> y <command>--with-custom-conf</command>. La opción --config del demonio GDM podría usarse también para especificar la ubicación del archivo de configuración. El demonio GDM debe reiniciarse para cambiar el archivo de configuración en uso.</para>
 
     <para><filename>&lt;share&gt;/gdm/factory-gdm.conf</filename> es el archivo de  configuración tal como se empaqueta con el demonio. Esto puede ser útil para ver si el archivo <filename>&lt;share&gt;/gdm/defaults.conf</filename> ha cambiado.</para>
 
@@ -388,18 +388,18 @@ PostSession/
 
     <para>Los módulos de accesibilidad se configuran en el subdirectorio <filename>modules/</filename>, y son un tema aparte. Lea los archivos predeterminados provistos, tienen documentación adecuada. De nuevo normalmente la instalación predeterminada se da en archivos con <filename>factory</filename> en su nombre, y dichos archivos no se leen, están solo ahí para que usted pueda volver siempre a la configuración predeterminada.</para>
 
-    <para>Files describing available GDM session follow the freedesktop.org desktop file specification and are <filename>.desktop</filename>-style files are installed to <filename>&lt;etc&gt;/X11/sessions/</filename>. This directory is also read by the KDE desktop manager (KDM) for common configuration. Next the directory <filename>&lt;share&gt;/gdm/BuiltInSessions/</filename> is read for GDM specific built-in sessions (KDM hardcodes these at time of this writing). Lastly the default setup will also read <filename>&lt;share&gt;/xsessions/</filename> (which should be <filename>&lt;share&gt;/xsessions/</filename> if you really wish to cooperate with KDM) where desktop packages can install their session files. The directories under the <filename>&lt;etc&gt;</filename> should be reserved for configuration. The desktop file specification approach makes it easy for package management systems to install window managers and different session types without requiring the sysadmin to edit files. See the <filename>SessionDesktopDir</filename> configuration key for changing the paths. It used to be that GDM stored its built in sessions in <filename>&lt;etc&gt;/dm/Sessions/</filename> but this is deprecated as of 2.5.90.0. Note that prior to version 2.4.4.2 only the <filename>&lt;etc&gt;/dm/Sessions/</filename> was being read.</para>
+    <para>Los archivos que describen las sesiones GDM disponibles siguen la especificación de archivos <filename>.desktop</filename> de freedesktop.org y se instalan en <filename>&lt;etc&gt;/X11/sessions/</filename>. Este directorio también lo lee el gestor de pantallas de KDE para una configuración común. El siguiente directorio <filename>&lt;share&gt;/gdm/BuiltInSessions/</filename> se lee para las sesiones integradas específicas de GDM (KDM las lleva en el propio código cuando se escribió este manual). Por último la configuración predeterminada también lee <filename>&lt;share&gt;/xsessions/</filename> (que debería ser  <filename>/usr/share/xsessions/</filename> si realmente quiere cooperar con KDM) donde los paquetes del escritorio pueden instalar sus propios archivos de sesión. Los directorios bajo <filename>&lt;etc&gt;</filename> deberían estar reservados para la configuración. La aproximación de la especificación de archivos de escritorio hace fácil para los sistemas de gestión de paquetes instalar gestores de ventanas y diferentes tipos de sesiones sin requerir que el administrador del sistema edite los archivos. Vea la clave de configuración <filename>SessionDesktopDir</filename> para cambiar las rutas. Solía ocurrir que GDM almacenaba sus sesiones integradas en <filename>&lt;etc&gt;/dm/Sessions/</filename> pero esto es obsoleto a partir de 2.5.90.0 Note que antes de la versión 2.4.4.2 sólo se leía <filename>&lt;etc&gt;/dm/Sessions/</filename>.</para>
 
-    <para>A session can be disabled (if it was installed in <filename>&lt;share&gt;/xsessions/</filename>) by adding an identically named <filename>.desktop</filename> to one of the directories earlier in the path (likely <filename>&lt;etc&gt;/X11/sessions</filename>) and using <filename>Hidden=true</filename> in that file.</para>
+    <para>Puede desactivarse una sesión (si fue instalada en <filename>&lt;share&gt;xsessions/</filename>) añadiendo un <filename>.desktop</filename> nombrado idénticamente a uno de los directorios anteriores en la ruta (como <filename>&lt;etc&gt;/X11/sessions</filename>) y usando <filename>Hidden=true</filename> en ese archivo.</para>
 
     <sect2 id="scriptdirs">
-      <title>Los directorio de script</title>
+      <title>Los directorio de scripts</title>
       
       <para>En esta sección explicaremos los directorios <filename>Init</filename>, <filename>PostLogin</filename>, <filename>PreSession</filename> y <filename>PostSession</filename> ya que son muy similares.</para>
 
       <para>Cuando el servidor X se haya iniciado con éxito, GDM intentará ejecutar el script llamado <filename>Init/&lt;displayname&gt;</filename>. Ej: <filename>Init/:0</filename> para la primera pantalla local. Si este archivo no se encuentra, GDM intentará ejecutar <filename>Init/&lt;hostname&gt;</filename>. Ej: <filename>Init/somehost</filename>. Si esto aún no se encuentra, GDM intentará <filename>Init/XDMCP</filename> para todas las entradas XDMCP o <filename>Init/Flexi</filename> para todas las pantallas bajo demanda. Si no se encuentra ninguno de los anteriores, GDM ejecutará <filename>Init/Default</filename>. El script se ejecutará como root y GDM se bloquea hasta que termina. Use el script <filename>Init/*</filename> para las aplicaciones que se suponen que se ejecutan junto con la ventana de entrada de GDM. xconsole por ejemplo. Los comandos para poner el fondo de pantalla, etc. van también en este archivo.</para>
 
-      <para>It is up to the sysadmin to decide whether clients started by the Init script should be killed before starting the user session. This is controlled with the <filename>KillInitClients</filename> configuration option.</para>
+      <para>Es responsabilidad del administrador de sistemas decidir si los clientes iniciados por el script Init deben matarse antes de iniciar la sesión del usuario. Esto se controla con la opción de configuración <filename>KillInitClients</filename></para>
 
       <para>When the user has been successfully authenticated GDM tries the scripts in the <filename>PostLogin</filename> directory in the same manner as for the <filename>Init</filename> directory. This is done before any session setup is done, and so this would be the script where you might setup the home directory if you need to (though you should use the <filename>pam_mount</filename> module if you can for this). You have the <filename>$USER</filename> and <filename>$DISPLAY</filename> environment variables set for this script, and again it is run as root. The script should return 0 on success as otherwise the user won't be logged in. This is not true for failsafe session however.</para>
 
@@ -419,9 +419,9 @@ PostSession/
     </sect2>
 
     <sect2 id="configfile">
-      <title>The Configuration Files - <filename>defaults.conf</filename> and <filename>custom.conf</filename></title>
+      <title>Los archivos de configuración - <filename>defaults.conf</filename> y <filename>custom.conf</filename></title>
       
-      <para>GDM uses two configuration files: <filename>&lt;share&gt;/gdm/defaults.conf</filename> and <filename>&lt;etc&gt;/gdm/custom.conf</filename>. The <filename>&lt;share&gt;/gdm/defaults.conf</filename> file contains the default configuration choices for GDM, and should not be modified by the user. The <filename>&lt;etc&gt;/gdm/custom.conf</filename> file is where users may specify their custom configuration choices. Configuration options specified in the <filename>&lt;etc&gt;/gdm/custom.conf</filename> file override the values in the <filename>&lt;share&gt;/gdm/defaults.conf</filename> file. If a configuration option is not defined in either file, GDM will default to the value described in the comments in the <filename>&lt;share&gt;/gdm/defaults.conf</filename> file.</para>
+      <para>GDM usa dos archivos de configuración: <filename>&lt;share&gt;/gdm/defaults.conf</filename> y <filename>&lt;etc&gt;/gdm/custom.conf</filename>. El archivo <filename>&lt;share&gt;/gdm/defaults.conf</filename> contiene las elecciones de configuración predeterminadas para GDM, y no debería modificarse por el usuario. El archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> es donde los usuarios pueden especificar sus elecciones de configuración personales. Las opciones de configuración especificadas en el archivo <filename>&lt;etc&gt;/gdm/custom.conf</filename> toman precedencia sobre los valores en el archivo <filename>&lt;share&gt;/gdm/defaults.conf</filename> Si una opción de configuración no está definida en cualquier archivo, GDM usará el valor predeterminado descrito en los comentarios en el archivo <filename>&lt;share&gt;/gdm/defaults.conf</filename>.</para>
 
       <para>Running the <command>gdmsetup</command> command will cause the <filename>&lt;etc&gt;/gdm/custom.conf</filename> to be modified with the user's configuration choices.</para>