Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(268)

Issue 408113003: Only build relocation packer for Arm and Arm64 (Closed)

Created:
6 years, 5 months ago by Anton
Modified:
6 years, 5 months ago
CC:
chromium-reviews
Project:
chromium
Visibility:
Public.

Description

Only build relocation packer for Arm and Arm64. Fix x64 build. BUG=346626

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M build/all.gyp View 2 chunks +5 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
Anton
The build for x64 was broken by: https://codereview.chromium.org/404553003 Both the CL description and the preprocessor ...
6 years, 5 months ago (2014-07-22 09:46:33 UTC) #1
simonb (inactive)
lgtm
6 years, 5 months ago (2014-07-22 09:58:32 UTC) #2
simonb (inactive)
On 2014/07/22 09:58:32, simonb wrote: > lgtm Oh, and thanks for fixing!
6 years, 5 months ago (2014-07-22 09:58:55 UTC) #3
rmcilroy
On 2014/07/22 09:58:55, simonb wrote: > On 2014/07/22 09:58:32, simonb wrote: > > lgtm > ...
6 years, 5 months ago (2014-07-22 12:35:40 UTC) #4
anton1
The CQ bit was checked by anton@google.com
6 years, 5 months ago (2014-07-22 12:35:52 UTC) #5
anton1
The CQ bit was unchecked by anton@google.com
6 years, 5 months ago (2014-07-22 12:37:22 UTC) #6
anton1
6 years, 5 months ago (2014-07-22 12:45:41 UTC) #7
On 2014/07/22 12:37:22, anton1 wrote:
> The CQ bit was unchecked by mailto:anton@google.com

Closing this one since the duplicate landed.

Powered by Google App Engine
This is Rietveld 408576698