For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.

For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64(both LE and BE).

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=adamk@chromium.org, dpranke@chromium.org, machenbach@chromium.org
BUG=706728
Committed: https://chromium.googlesource.com/chromium/deps/icu/+/c5f212bac791598c3367d9318c3452b47944cf54

[rayb, 2017-04-12 01:35:34]

Description was changed from

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.
Also added support for host_byteorder logic.

Work in progress.

Related to the v8_issue=

BUG=706728
==========

to

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.
Also added support for host_byteorder logic.

Work in progress.

Related to the v8_issue=2809963004

BUG=706728
==========

[rayb, 2017-04-12 17:00:31]

Description was changed from

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.
Also added support for host_byteorder logic.

Work in progress.

Related to the v8_issue=2809963004

BUG=706728
==========

to

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.

Also added support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, agrieve@chromium.org
BUG=706728
==========

[rayb, 2017-04-12 17:01:50]

Description was changed from

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.

Also added support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, agrieve@chromium.org
BUG=706728
==========

to

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, agrieve@chromium.org
BUG=706728
==========

[rayb, 2017-04-12 17:10:59]

Description was changed from

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64.

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, agrieve@chromium.org
BUG=706728
==========

to

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64(both LE and
BE).

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, agrieve@chromium.org
BUG=706728
==========

[rayb, 2017-04-12 17:15:31]

Description was changed from

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64(both LE and
BE).

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, agrieve@chromium.org
BUG=706728
==========

to

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64(both LE and
BE).

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, adamk@chromium.org
BUG=706728
==========

[rayb, 2017-04-12 17:24:40]

rayb@ca.ibm.com changed reviewers:
+ adamk@chromium.org, dpranke@chromium.org, machenbach@chromium.org

[rayb, 2017-04-12 17:24:40]

ptal

[Dirk Pranke, 2017-04-14 01:32:49]

lgtm

[Michael Achenbach, 2017-04-18 13:58:18]

lgtm

[rayb, 2017-04-25 00:42:14]

ptal.

Uses host_byteorder.gni now.


Also updated the three related cls -

https://codereview.chromium.org/2807463004/
https://codereview.chromium.org/2809963004/
https://codereview.chromium.org/2815453004/

[Dirk Pranke, 2017-04-25 01:38:16]

lgtm

[rayb, 2017-05-14 17:02:36]

On 2017/04/25 01:38:16, Dirk Pranke (slow) wrote:
> lgtm

It seems I'm unable to commit the patch using 'git cl land'. It generates the
following output - 

Review-Url: https://codereview.chromium.org/2812173002 .
 BUILD.gn | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
Attempt 1 of 3
Fetching origin/refs/heads/master...
From https://chromium.googlesource.com/chromium/deps/icu
 * [new branch]      master     -> git-cl-cherry-pick
Cherry-picking commit on top of latest refs/heads/master
Counting objects: 5, done.
Delta compression using up to 120 threads.
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 532 bytes | 0 bytes/s, done.
Total 3 (delta 2), reused 0 (delta 0)
remote: Resolving deltas: 100% (2/2)
remote: Processing changes: refs: 1, done    
error: failed to push some refs to
'https://chromium.googlesource.com/chromium/deps/icu'
To https://chromium.googlesource.com/chromium/deps/icu
!	HEAD:refs/heads/master	[remote rejected] (prohibited by Gerrit: ref update
access denied)
Done

I should be able to push from a publicly logged-in account (
https://chromium-review.googlesource.com/?polygerrit=0#/admin/projects/chromi...
).

Not quite sure what I'm doing wrong. Any help help would be greatly appreciated.

[Michael Achenbach, 2017-05-15 06:37:28]

Description was changed from

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64(both LE and
BE).

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=machenbach@chromium.org, dpranke@chromium.org, adamk@chromium.org
BUG=706728
==========

to

==========
For building v8 using gn on aix_ppc64, linux_s390x and linux_ppc64(both LE and
BE).

Also add support for host_byteorder logic which is introduced in -
https://codereview.chromium.org/2815453004/

R=adamk@chromium.org, dpranke@chromium.org, machenbach@chromium.org
BUG=706728

Committed:
https://chromium.googlesource.com/chromium/deps/icu/+/c5f212bac791598c3367d93...
==========

[Michael Achenbach, 2017-05-15 06:37:29]

Committed patchset #4 (id:60001) manually as
c5f212bac791598c3367d9318c3452b47944cf54 (presubmit successful).

[Michael Achenbach, 2017-05-15 06:38:05]

Landed it for you. Guess you need committer access.