Allow multiple OSes to be checked out with git submodules.

Allow multiple OSes to be checked out with git submodules.

Allow target.os to be a multivalue config option, which will cause
submodules required by any of the listed OSes to be checked out. You can
add additional OSes to the list with "git config --add target.os".

BUG=

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

[Torne, 2013-04-11 10:10:07]

Hi Stefan,

I wanted to be able to build linux and android chromium from the same checkout,
so I wrote this, which seems to work and not break existing configs.

[szager1, 2013-04-11 17:26:03]

lgtm with small change.

Thanks!

https://codereview.chromium.org/13831004/diff/1/crup-runner.sh
File crup-runner.sh (right):

https://codereview.chromium.org/13831004/diff/1/crup-runner.sh#newcode68
crup-runner.sh:68: if [ "$update_policy" == checkout ]; then
Let's make this:

if [ "$update_policy" != "none" ]; then

There are other valid values for "update" which could be set by the user.

[Torne, 2013-04-12 10:04:36]

https://codereview.chromium.org/13831004/diff/1/crup-runner.sh
File crup-runner.sh (right):

https://codereview.chromium.org/13831004/diff/1/crup-runner.sh#newcode68
crup-runner.sh:68: if [ "$update_policy" == checkout ]; then
 2013/04/11 17:26:03, szager1 wrote:
> Let's make this:
> 
> if [ "$update_policy" != "none" ]; then
> 
> There are other valid values for "update" which could be set by the user.

This can't actually happen, because this is inside the if block for -z
$update_policy, i.e. this all only happens if the configuration didn't exist
before. I'll change it anyway, though, since it seems clearer that way.

[commit-bot: I haz the power, 2013-04-12 10:04:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/torne@chromium.org/13831004/4001

[commit-bot: I haz the power, 2013-04-12 10:06:48]

Change committed as 193901