Add ability to blacklist packages from chromeos-base/chromeos.

Add ability to blacklist packages from chromeos-base/chromeos.

Change-Id: I05370137dbe9f9adc7b18f2ff77d1b7680275c37

BUG=7973
TEST=Ran with/without current blacklist file.  Also, added chromeos-base/chromeos-chrome
to test it actually dies correctly.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=e38f90f

[sosa, 2010-10-20 22:24:51]

Base blacklist includes java & python

[petkov, 2010-10-20 22:36:53]

LGTM w/ a couple of optional suggestions. Thanks!

http://codereview.chromium.org/4005002/diff/1/2
File build_image (right):

http://codereview.chromium.org/4005002/diff/1/2#newcode109
build_image:109: local package_list=$(${SCRIPTS_DIR}/get_package_list \
have you tried/considered something like:

get_package_list | grep -x -f ${package_blacklist_file}

not sure if it works :)

http://codereview.chromium.org/4005002/diff/1/2#newcode112
build_image:112: if echo "${package_list}" | egrep ^${package}$ 2>1 > /dev/null;
then
you can do grep -x ${package} here, I think.

[sosa, 2010-10-20 23:07:15]

Done and pushed.

On Wed, Oct 20, 2010 at 3:36 PM,  <petkov@chromium.org> wrote:
> LGTM w/ a couple of optional suggestions. Thanks!
>
>
>
> http://codereview.chromium.org/4005002/diff/1/2
> File build_image (right):
>
> http://codereview.chromium.org/4005002/diff/1/2#newcode109
> build_image:109: local package_list=$(${SCRIPTS_DIR}/get_package_list \
> have you tried/considered something like:
>
> get_package_list | grep -x -f ${package_blacklist_file}
>
> not sure if it works :)
>
> http://codereview.chromium.org/4005002/diff/1/2#newcode112
> build_image:112: if echo "${package_list}" | egrep ^${package}$ 2>1 >
> /dev/null; then
> you can do grep -x ${package} here, I think.
>
> http://codereview.chromium.org/4005002/show
>