www-apps/postfixadmin: Replace path_exists with inline logic

Replace the call to path_exists with local filename expansion.  While
at it, also kill the ugly ls-abuse in favor of using the results
of earlier expansion.

diff --git a/www-apps/postfixadmin/postfixadmin-3.1.ebuild b/www-apps/postfixadmin/postfixadmin-3.1.ebuild
index 3e69bc6b6573452b37fd1a0833378dcd7e40d439..5cb58aab877a224ce725d0406a5087dec8370ec6 100644 (file)
--- a/www-apps/postfixadmin/postfixadmin-3.1.ebuild
+++ b/www-apps/postfixadmin/postfixadmin-3.1.ebuild
@@ -3,7 +3,7 @@
 
 EAPI="6"
 
-inherit eutils user webapp
+inherit user webapp
 
 DESCRIPTION="Web Based Management tool for Postfix style virtual domains and users"
 HOMEPAGE="http://postfixadmin.sourceforge.net"
@@ -89,10 +89,12 @@ pkg_postinst() {
 pkg_postrm() {
        # Make sure we don't leave broken vacation.pl symlink
        find -L "${ROOT}"/var/spool/vacation/ -type l -delete
-       if [[ ! -e "${ROOT}"/var/spool/vacation/vacation.pl ]] &&
-               path_exists "${ROOT}"/var/spool/vacation/vacation.pl-*; then
-               ln -s $(LC_ALL=C ls -1 /var/spool/vacation/vacation.pl-* | tail -n1) \
-                       "${ROOT}"/var/spool/vacation/vacation.pl || die
+       local shopt_save=$(shopt -p nullglob)
+       shopt -s nullglob
+       local vacation=( "${ROOT}"/var/spool/vacation/vacation.pl-* )
+       ${shopt_save}
+       if [[ ! -e "${ROOT}"/var/spool/vacation/vacation.pl && -n ${vacation[@]} ]]; then
+               ln -s "${vacation[-1]}" "${ROOT}"/var/spool/vacation/vacation.pl || die
                ewarn "/var/spool/vacation/vacation.pl was updated to point on most"
                ewarn "recent verion, but please, do your own checks"
        fi