Remove psutil from tree, install via install-build-deps.sh

Remove psutil from tree, install via install-build-deps.sh

Per discussion with thakis, cmp we should remove this dep from the tree and instead install it via install-build-deps.sh.

BUG=106105
TEST=Installed dep, ran tests.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114146

[DaleCurtis, 2011-12-13 00:00:25]

PTAL. Follow up from http://codereview.chromium.org/8774018/

[Nico, 2011-12-13 00:05:31]

I like this much better :-)

Be sure to inform the infra guys that this needs to be installed on the existing
bugs. If you talked to cmp, I assume he's aware of that.

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh
File build/install-build-deps.sh (right):

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh#new...
build/install-build-deps.sh:141: plugin_list="flashplugin-installer
python-psutil"
plugin_list seems wrong…does adding it to dev_list above work too?

[DaleCurtis, 2011-12-13 00:06:56]

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh
File build/install-build-deps.sh (right):

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh#new...
build/install-build-deps.sh:141: plugin_list="flashplugin-installer
python-psutil"
On 2011/12/13 00:05:31, Nico wrote:
> plugin_list seems wrong…does adding it to dev_list above work too?

They all funnel into a apt-get install call below. I chose this one because it's
titled '... for tests.' Happy to move it else where.

[Nico, 2011-12-13 00:10:35]

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh
File build/install-build-deps.sh (right):

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh#new...
build/install-build-deps.sh:141: plugin_list="flashplugin-installer
python-psutil"
On 2011/12/13 00:06:56, DaleCurtis wrote:
> They all funnel into a apt-get install call below. I chose this one because
it's
> titled '... for tests.' Happy to move it else where.

It's also titled "Plugin list", and python modules aren't plugins :-) Since
tests are part of development, I think dev_list is a slightly better fit.

[DaleCurtis, 2011-12-13 00:12:42]

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh
File build/install-build-deps.sh (right):

http://codereview.chromium.org/8919026/diff/1/build/install-build-deps.sh#new...
build/install-build-deps.sh:141: plugin_list="flashplugin-installer
python-psutil"
Fair enough :) Done.

On 2011/12/13 00:10:35, Nico wrote:
> On 2011/12/13 00:06:56, DaleCurtis wrote:
> > They all funnel into a apt-get install call below. I chose this one because
> it's
> > titled '... for tests.' Happy to move it else where.
> 
> It's also titled "Plugin list", and python modules aren't plugins :-) Since
> tests are part of development, I think dev_list is a slightly better fit.

[Nico, 2011-12-13 00:14:06]

LGTM, thanks!

http://codereview.chromium.org/8919026/diff/6001/build/install-build-deps.sh
File build/install-build-deps.sh (right):

http://codereview.chromium.org/8919026/diff/6001/build/install-build-deps.sh#...
build/install-build-deps.sh:115: libcurl4-gnutls-dev ttf-indic-fonts
ttf-thai-tlwg python-psutil
nit: It looks like this list is alphabetically sorted, so maybe insert this
after python-dev instead

[DaleCurtis, 2011-12-13 00:35:02]

http://codereview.chromium.org/8919026/diff/6001/build/install-build-deps.sh
File build/install-build-deps.sh (right):

http://codereview.chromium.org/8919026/diff/6001/build/install-build-deps.sh#...
build/install-build-deps.sh:115: libcurl4-gnutls-dev ttf-indic-fonts
ttf-thai-tlwg python-psutil
On 2011/12/13 00:14:06, Nico wrote:
> nit: It looks like this list is alphabetically sorted, so maybe insert this
> after python-dev instead

Well it wasn't quite sorted, so I fixed that, although this diff is now pretty
dirty. :( I checked I didn't drop any packages:

for pkg in ${dev_list_orig}; do echo $pkg; done | sort > a1.txt
for pkg in ${dev_list_new}; do echo $pkg; done | sort > a2.txt
diff a1.txt a2.txt

[commit-bot: I haz the power, 2011-12-13 00:35:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/8919026/4002

[commit-bot: I haz the power, 2011-12-13 01:42:51]

Change committed as 114146