Don't run simplechrome hooks if checkout_src_internal is true.

The hook for checkout_src_internal needs some tweaking to its args. Adding
those to src-internal's DEPS in crrev.com/i/1321692

This assumes that if you have checkout_src_internal set to true, you don't
want to build or test on publicly-available boards. That's not necessarily
true, since I could checkout src-internal for unrelated reasons (eg: for
android-internal development) but still only want to do chromeos-public
development. This is unfortunate, but not terribly problematic because:

- These simplechrome gclient hooks exist only to get the automated builds
(ie: those on bots) to fetch all their inputs in a single step. A human
developer can simply rerun the cros-chrome-sdk with their own options.

- The assumption that if you have checkout_src_internal set to true, you don't
want to build or test on publicly-available boards will *always* be true for bots.
(ie: bots will only ever build for a single board+configuration). In other words,
a bot will always either want to run on a private board w/ src-internal, or a
public board w/ just src.

- This adds a third gclient var (use_public_cros_config) which can control this
behavior explicitly, but defaults to 'not checkout_src_internal'.

Bug: 947531
Change-Id: I23e513e0dbfeaaefa92044012866f3d34fe6ee57
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1622431
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#661897}
1 file changed