Improving parallelism when building with goma

Improving parallelism when building with goma

It seems that ninja defaults the number of available CPUs.

BUG=chromium:690916
NOTRY=True

Review-Url: https://codereview.webrtc.org/2691393003
Cr-Commit-Position: refs/heads/master@{#16635}
Committed: https://chromium.googlesource.com/external/webrtc/+/8714b8f1aecbf04eec90d1fc86f151072d629120

[kjellander_webrtc, 2017-02-15 14:40:58]

kjellander@webrtc.org changed reviewers:
+ kjellander@webrtc.org

[kjellander_webrtc, 2017-02-15 14:40:59]

huh, is the trybot not building anything?!

https://codereview.webrtc.org/2691393003/diff/1/tools-webrtc/ios/build_ios_li...
File tools-webrtc/ios/build_ios_libs.py (right):

https://codereview.webrtc.org/2691393003/diff/1/tools-webrtc/ios/build_ios_li...
tools-webrtc/ios/build_ios_libs.py:127: cmd.extend(['j', '200'])
Should be -j

[mbonadei, 2017-02-15 14:43:25]

On 2017/02/15 14:40:59, kjellander_webrtc wrote:
> huh, is the trybot not building anything?!
> 
>
https://codereview.webrtc.org/2691393003/diff/1/tools-webrtc/ios/build_ios_li...
> File tools-webrtc/ios/build_ios_libs.py (right):
> 
>
https://codereview.webrtc.org/2691393003/diff/1/tools-webrtc/ios/build_ios_li...
> tools-webrtc/ios/build_ios_libs.py:127: cmd.extend(['j', '200'])
> Should be -j

Yeah, good catch!

Yep, that is strange.

[kjellander_webrtc, 2017-02-15 14:46:47]

lgtm, but let's wait for the recipe to be fixed first, then do another try job
run.

[mbonadei, 2017-02-15 21:15:56]

The CQ bit was checked by mbonadei@webrtc.org

[commit-bot: I haz the power, 2017-02-15 21:16:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-02-15 21:18:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487193356870190,
"parent_rev": "68ab366547b7494b69e28047b17ad2ebd64e9edd", "commit_rev":
"8714b8f1aecbf04eec90d1fc86f151072d629120"}

[commit-bot: I haz the power, 2017-02-15 21:19:02]

Description was changed from

==========
Improving parallelism when building with goma

It seems that ninja defaults the number of available CPUs.

BUG=chromium:690916
NOTRY=True
==========

to

==========
Improving parallelism when building with goma

It seems that ninja defaults the number of available CPUs.

BUG=chromium:690916
NOTRY=True

Review-Url: https://codereview.webrtc.org/2691393003
Cr-Commit-Position: refs/heads/master@{#16635}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8714b8f1aecbf04eec90d1fc8...
==========

[commit-bot: I haz the power, 2017-02-15 21:19:03]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/8714b8f1aecbf04eec90d1fc8...