Merge lp:~bac/launchpad/bug-637102 into lp:launchpad

Proposed by Brad Crittenden
Status: Merged
Approved by: Aaron Bentley
Approved revision: no longer in the source branch.
Merged at revision: 11580
Proposed branch: lp:~bac/launchpad/bug-637102
Merge into: lp:launchpad
Diff against target: 21 lines (+2/-2)
1 file modified
utilities/rocketfuel-setup (+2/-2)
To merge this branch: bzr merge lp:~bac/launchpad/bug-637102
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Review via email: mp+36028@code.launchpad.net

Commit message

Use apt-get instead of aptitude in rocketfuel-setup as aptitude is no longer installed by default.

Description of the change

In Ubuntu 10.10, aptitude is not installed by default. We could either add it to lp-dev-dependencies or just use apt-get. I chose the latter.

To post a comment you must log in.
Revision history for this message
Aaron Bentley (abentley) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'utilities/rocketfuel-setup'
2--- utilities/rocketfuel-setup 2010-08-06 17:39:52 +0000
3+++ utilities/rocketfuel-setup 2010-09-20 15:22:43 +0000
4@@ -217,7 +217,7 @@
5 dpkg -s $pkg | grep -q "^Status: install ok installed"
6 if [ $? -ne 0 ]; then
7 echo "Installing $pkg package..."
8- sudo aptitude install $pkg
9+ sudo apt-get install $pkg
10 if [ $? -ne 0 ]; then
11 echo "Unable to install $pkg."
12 exit 1
13@@ -225,7 +225,7 @@
14 fi
15 }
16
17-sudo aptitude update
18+sudo apt-get update
19 REQUIRED_PACKAGES="launchpad-developer-dependencies apache2 apache2-mpm-worker"
20 for pkg in $REQUIRED_PACKAGES; do
21 do_install;