[gentoo-commits] eselect r516 - in trunk: . modules

[gentoo-commits] eselect r516 - in trunk: . modules

[Ulrich Mueller (ulm)]

Author: ulm
Date: 2009-04-28 11:34:56 +0000 (Tue, 28 Apr 2009)
New Revision: 516

Modified:
   trunk/ChangeLog
   trunk/NEWS
   trunk/modules/mailer.eselect
Log:
Highlight selected target in mailer module, bug 220473.

Modified: trunk/ChangeLog
===================================================================
--- trunk/ChangeLog	2009-04-28 09:55:27 UTC (rev 515)
+++ trunk/ChangeLog	2009-04-28 11:34:56 UTC (rev 516)
@@ -1,5 +1,8 @@
 2009-04-28  Ulrich Mueller  <ulm@gentoo.org>
 
+	* modules/mailer.eselect (do_list): Highlight selected target,
+	patch by Karl Hakimian <t4y68ds02@sneakemail.com> in 220473.
+
 	* configure.ac: Fix test for greadlink.
 
 	* configure.ac: Add --with-pm option for configuring the preferred

Modified: trunk/NEWS
===================================================================
--- trunk/NEWS	2009-04-28 09:55:27 UTC (rev 515)
+++ trunk/NEWS	2009-04-28 11:34:56 UTC (rev 516)
@@ -6,6 +6,7 @@
     - Fixed bug #155814: Clear aliases and shell functions.
     - Fixed bug #156866: Handle missing scripts in rc module.
     - Fixed bug #180966: Make rc module work with OpenRC.
+    - Fixed bug #220473: Highlight selected target in mailer module.
     - Various improvements in opengl module.
 
     New features:

Modified: trunk/modules/mailer.eselect
===================================================================
--- trunk/modules/mailer.eselect	2009-04-28 09:55:27 UTC (rev 515)
+++ trunk/modules/mailer.eselect	2009-04-28 11:34:56 UTC (rev 516)
@@ -1,4 +1,4 @@
-# Copyright 1999-2005 Gentoo Foundation
+# Copyright 1999-2009 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
 # $Id$
 
@@ -83,14 +83,15 @@
 		for (( i = 0 ; i < ${#targets[@]} ; i = i + 1 )) ; do
 			# any more elegant way to do this?
 			local profile
-			profile="$(canonicalise ${ROOT}/etc/mail/mailer.conf )"
-			[[ ${targets[${i}]} == ${profile%.mailer} ]] && \
-				targets[${i}]="${targets[${i}]} $(highlight '*' )"
-			done
-			write_numbered_list "${targets[@]}"
-		else
-			write_kv_list_entry "(none found)" ""
-		fi
+			profile=$(basename \
+				"$(canonicalise "${ROOT}/etc/mail/mailer.conf")")
+			[[ ${targets[i]} = ${profile%.mailer} ]] \
+				&& targets[i]="${targets[i]} $(highlight '*')"
+		done
+		write_numbered_list "${targets[@]}"
+	else
+		write_kv_list_entry "(none found)" ""
+	fi
 }
 
 ### set action ###