Linux ARM Cross-Compile recipe

Linux ARM Cross-Compile recipe

Converted Linux ARM Cross-Compile builder on chromium.fyi master to a recipe.

R=phajdan.jr@chromium.org
BUG=359338
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=272598

[nodir, 2014-05-23 01:24:21]

https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
File scripts/slave/recipes/chromium.py (right):

https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
scripts/slave/recipes/chromium.py:474: 'chromium_swarm_tests',
Pawel, can you confirm that this will compile browser_tests?

[Paweł Hajdan Jr., 2014-05-23 08:49:45]

LGTM with comments

https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
File scripts/slave/recipes/chromium.py (right):

https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
scripts/slave/recipes/chromium.py:474: 'chromium_swarm_tests',
On 2014/05/23 01:24:22, nodir wrote:
> Pawel, can you confirm that this will compile browser_tests?

If you want to be sure please add them explicitly - make the intent clear and
avoid assumptions that may get invalidated.

Please see src/build/all.gyp : chromium_swarm_tests will compile browser_tests,
but needs test_isolation_mode != noop. It seems this should be used if you want
to use swarm.

And if you want to compile browser_tests, use "browser_tests" or "all".

These chromium_* compile targets were created before recipes, so that one could
change what they build without restarting the master.

[nodir, 2014-05-23 21:45:10]

On 2014/05/23 08:49:45, Paweł Hajdan Jr. wrote:
> LGTM with comments
> 
>
https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
> File scripts/slave/recipes/chromium.py (right):
> 
>
https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
> scripts/slave/recipes/chromium.py:474: 'chromium_swarm_tests',
> On 2014/05/23 01:24:22, nodir wrote:
> > Pawel, can you confirm that this will compile browser_tests?
> 
> If you want to be sure please add them explicitly - make the intent clear and
> avoid assumptions that may get invalidated.
> 
> Please see src/build/all.gyp : chromium_swarm_tests will compile
browser_tests,
> but needs test_isolation_mode != noop. It seems this should be used if you
want
> to use swarm.
> 
> And if you want to compile browser_tests, use "browser_tests" or "all".
> 
> These chromium_* compile targets were created before recipes, so that one
could
> change what they build without restarting the master.

Since the actual list of tests that needs to be compiled is specified in
chromium_arm.json test spec file, I've specified "all".

[nodir, 2014-05-23 21:46:32]

The CQ bit was checked by nodir@chromium.org

[commit-bot: I haz the power, 2014-05-23 21:47:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nodir@chromium.org/298653008/20001

[commit-bot: I haz the power, 2014-05-23 21:48:40]

Change committed as 272598

[nodir, 2014-05-27 19:17:15]

https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
File scripts/slave/recipes/chromium.py (right):

https://codereview.chromium.org/298653008/diff/1/scripts/slave/recipes/chromi...
scripts/slave/recipes/chromium.py:474: 'chromium_swarm_tests',
On 2014/05/23 08:49:46, Paweł Hajdan Jr. wrote:
> On 2014/05/23 01:24:22, nodir wrote:
> > Pawel, can you confirm that this will compile browser_tests?
> 
> If you want to be sure please add them explicitly - make the intent clear and
> avoid assumptions that may get invalidated.
> 
> Please see src/build/all.gyp : chromium_swarm_tests will compile
browser_tests,
> but needs test_isolation_mode != noop. It seems this should be used if you
want
> to use swarm.
> 
> And if you want to compile browser_tests, use "browser_tests" or "all".
> 
> These chromium_* compile targets were created before recipes, so that one
could
> change what they build without restarting the master.

Do we really need to specify targets here? They seem to be read from test spec
file as well, which specifies the exact targets we need.