Add unsupported option to install-build-deps.

Add unsupported option to install-build-deps.

Add an option --unsupported to install-build-deps.sh which allows the
installation to be attempted on a system we don't support. Omit
mentioning this in the error, in the hope that anyone who can deal with
the fallout from this will find it in the script. Also, add GCEL
(Ubuntu-derived image used by Compute Engine) to the list of supported
systems.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176982

[Torne, 2013-01-11 17:07:40]

Hi Tony,

Is it okay to add this option to make my life easier automating stuff on Compute
Engine (which advertises itself as "GCEL" instead of Ubuntu, though it's still
Ubuntu-based)? In general I would think anyone who is going to read the script
to discover the option is able to just delete the check. :)

[tony, 2013-01-11 18:20:01]

Can we instead update the regex to allow Ubuntu|GCEL?  What is in /etc/issue on
Compute Engine?

[Torne, 2013-01-14 16:51:28]

On 2013/01/11 18:20:01, tony wrote:
> Can we instead update the regex to allow Ubuntu|GCEL?  What is in /etc/issue
on
> Compute Engine?

Compute Engine's issue says:
GCEL %s LTS \
 \l

It seems like this is a bug in the VM image, the %s should probably be  the
version number :) So, we could allow "GCEL" without checking the version (I'll
see if I can fix GCE to get the version in there, though).

If you'd rather have that than --unsupported that's fine, I can change the
patch, but it seems like it's a reasonable thing to have (for the benefit of
newly released Ubuntu versions, or maybe Debian)? Up to you.

[tony, 2013-01-14 17:49:06]

I like updating the regex because it makes people who are testing the script
make sure it still runs on newer/different versions of Ubuntu (sometimes
packages get renamed).  For example, if you add GCEL the next person to try
running on Compute Engine doesn't have to wonder if the script will run
properly.

[Torne, 2013-01-14 18:11:26]

On 2013/01/14 17:49:06, tony wrote:
> I like updating the regex because it makes people who are testing the script
> make sure it still runs on newer/different versions of Ubuntu (sometimes
> packages get renamed).  For example, if you add GCEL the next person to try
> running on Compute Engine doesn't have to wonder if the script will run
> properly.

That's a good point, but maybe we could do both?

[tony, 2013-01-14 18:15:04]

On 2013/01/14 18:11:26, Torne wrote:
> On 2013/01/14 17:49:06, tony wrote:
> > I like updating the regex because it makes people who are testing the script
> > make sure it still runs on newer/different versions of Ubuntu (sometimes
> > packages get renamed).  For example, if you add GCEL the next person to try
> > running on Compute Engine doesn't have to wonder if the script will run
> > properly.
> 
> That's a good point, but maybe we could do both?

I kind of want people who pass --unsupported to contribute their knowledge back
to this script, but maybe that's wishful thinking on my part.  Adding both
--unsupported and adding GCEL to the regex sounds fine to me.

[Torne, 2013-01-15 21:59:50]

On 2013/01/14 18:15:04, tony wrote:
> On 2013/01/14 18:11:26, Torne wrote:
> > On 2013/01/14 17:49:06, tony wrote:
> > > I like updating the regex because it makes people who are testing the
script
> > > make sure it still runs on newer/different versions of Ubuntu (sometimes
> > > packages get renamed).  For example, if you add GCEL the next person to
try
> > > running on Compute Engine doesn't have to wonder if the script will run
> > > properly.
> > 
> > That's a good point, but maybe we could do both?
> 
> I kind of want people who pass --unsupported to contribute their knowledge
back
> to this script, but maybe that's wishful thinking on my part.  Adding both
> --unsupported and adding GCEL to the regex sounds fine to me.

Updated patch to do both.

[tony, 2013-01-15 22:01:59]

LGTM, thanks!

https://codereview.chromium.org/11863012/diff/8001/build/install-build-deps.sh
File build/install-build-deps.sh (right):

https://codereview.chromium.org/11863012/diff/8001/build/install-build-deps.s...
build/install-build-deps.sh:45: # the're doing.
Nit: the're -> they're

[Torne, 2013-01-15 22:15:06]

On 2013/01/15 22:01:59, tony wrote:
> LGTM, thanks!
> 
> https://codereview.chromium.org/11863012/diff/8001/build/install-build-deps.sh
> File build/install-build-deps.sh (right):
> 
>
https://codereview.chromium.org/11863012/diff/8001/build/install-build-deps.s...
> build/install-build-deps.sh:45: # the're doing.
> Nit: the're -> they're

Fixed and landed.