[gn] Port linux_use_bundled_binutils_override GN variable

[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
NOTRY=true

Committed: https://crrev.com/d4ec5f1c256e99ff7cedb55030a8291b5e5fabb0
Cr-Commit-Position: refs/heads/master@{#39034}

[Michael Achenbach, 2016-08-31 06:47:57]

Description was changed from

==========
[gn] Port linux_use_bundled_binutils_override GN variable

BUG=
==========

to

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
==========

[Michael Achenbach, 2016-08-31 06:47:58]

machenbach@chromium.org changed reviewers:
+ jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-08-31 06:48:11]

Blocks https://codereview.chromium.org/2293333002/

[Michael Achenbach, 2016-08-31 06:48:20]

Description was changed from

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
==========

to

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

[Michael Achenbach, 2016-08-31 06:48:32]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-08-31 06:48:43]

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

[commit-bot: I haz the power, 2016-08-31 06:50:20]

Description was changed from

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

to

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

[commit-bot: I haz the power, 2016-08-31 06:50:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-31 06:50:39]

Description was changed from

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

to

==========
[gn] Port linux_use_bundled_binutils_override GN variable

Ports https://codereview.chromium.org/2293853002 to unblock
deps roller.

BUG=474921
TBR=jochen@chromium.org, kjellander@chromium.org, vogelheim@chromium.org
NOTRY=true

Committed: https://crrev.com/d4ec5f1c256e99ff7cedb55030a8291b5e5fabb0
Cr-Commit-Position: refs/heads/master@{#39034}
==========

[commit-bot: I haz the power, 2016-08-31 06:50:39]

Patchset 1 (id:??) landed as
https://crrev.com/d4ec5f1c256e99ff7cedb55030a8291b5e5fabb0
Cr-Commit-Position: refs/heads/master@{#39034}

[Lei Zhang, 2016-08-31 07:41:37]

thestig@chromium.org changed reviewers:
+ thestig@chromium.org

[Lei Zhang, 2016-08-31 07:41:38]

lgtm

https://codereview.chromium.org/2290233005/diff/1/build_overrides/build.gni
File build_overrides/build.gni (right):

https://codereview.chromium.org/2290233005/diff/1/build_overrides/build.gni#n...
build_overrides/build.gni:20: # Some non-Chromium builds don't use Chromium's
third_party/binutils.
This should say "V8 uses Chromium's third_party/binutils to speed up linking."

[Michael Achenbach, 2016-08-31 08:31:09]

https://codereview.chromium.org/2290233005/diff/1/build_overrides/build.gni
File build_overrides/build.gni (right):

https://codereview.chromium.org/2290233005/diff/1/build_overrides/build.gni#n...
build_overrides/build.gni:20: # Some non-Chromium builds don't use Chromium's
third_party/binutils.
On 2016/08/31 07:41:37, Lei Zhang wrote:
> This should say "V8 uses Chromium's third_party/binutils to speed up linking."

Hmm, yea - I didn't tidy any of the comments here. Feel free to change it. I
filed http://crbug.com/642641 for the general probelm.