[asterisk-commits] mvanbaak: trunk r208542 - /trunk/contrib/scripts/install_prereq

[SVN commits to the Asterisk project]

Author: mvanbaak
Date: Fri Jul 24 09:35:49 2009
New Revision: 208542

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=208542
Log:
use aptitude for debian based systems

The function to check wether we need to install packages was using
dpkg-query which was gives wrong output on Debian 5

Also, the apt-get has been replaced with aptitude because aptitude
is now the preferred way to handle packages on Debian

(closes issue #15570)
Reported by: mvanbaak
Patches:
      2009072400_installprereq-aptitude.diff uploaded by mvanbaak (license 7)

Modified:
    trunk/contrib/scripts/install_prereq

Modified: trunk/contrib/scripts/install_prereq
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/contrib/scripts/install_prereq?view=diff&rev=208542&r1=208541&r2=208542
==============================================================================
--- trunk/contrib/scripts/install_prereq (original)
+++ trunk/contrib/scripts/install_prereq Fri Jul 24 09:35:49 2009
@@ -45,10 +45,9 @@
   test "$testcmd" != ''
 }
 
-# Fixme: should be done by apt and not by dpkg?
 check_installed_debs() {
-	 dpkg-query -W --showformat '${Status} ${Package}\n' "$@" 2>/dev/null \
-	 | awk '/ not-installed/{print $4}'
+	aptitude -F '%c %p' search "$@" 2>/dev/null \
+	| awk '/^p/{print $2}'
 }
 
 # parsing the output of yum is close to impossible.
@@ -65,7 +64,7 @@
 handle_debian() {
 	# echo "# Distribution is Debian or compatible"
 	extra_packs=`check_installed_debs $PACKAGES_DEBIAN`
-	$testcmd apt-get install -y $extra_packs
+	$testcmd aptitude install -y $extra_packs
 }
 
 handle_rh() {