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

Issue 439533004: Port most sync targets to sync/BUILD.gn (Closed)

Created:
6 years, 4 months ago by rlarocque
Modified:
6 years, 4 months ago
CC:
chromium-reviews, tim+watch_chromium.org, haitaol+watch_chromium.org, zea+watch_chromium.org, maniscalco+watch_chromium.org
Project:
chromium
Visibility:
Public.

Description

Port most sync targets to sync/BUILD.gn Ports most of the sync GYP targets to GN. Adds comments in the original GYP files pointing to the new GN targets. Fixes some TODOs in other files that were blocked on sync GN support. Adds several new linkable targets to the GN build. The android-specific parts of sync have not been converted yet. TBR=nyquist,rogerta BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287905

Patch Set 1 #

Total comments: 1

Patch Set 2 : Rebase #

Patch Set 3 : Rebase again #

Unified diffs Side-by-side diffs Delta from patch set Stats (+782 lines, -8 lines) Patch
M BUILD.gn View 1 1 chunk +1 line, -0 lines 0 comments Download
M components/dom_distiller/webui/BUILD.gn View 1 chunk +1 line, -1 line 0 comments Download
M components/invalidation/BUILD.gn View 1 1 chunk +0 lines, -1 line 0 comments Download
M components/sync_driver/BUILD.gn View 1 chunk +1 line, -1 line 0 comments Download
M google_apis/BUILD.gn View 2 chunks +3 lines, -1 line 0 comments Download
M sync/BUILD.gn View 1 2 1 chunk +768 lines, -4 lines 0 comments Download
M sync/sync_tests.gypi View 1 2 8 chunks +8 lines, -0 lines 0 comments Download

Messages

Total messages: 23 (0 generated)
rlarocque
Here's the patch to support building sync with GN on Linux. I'll try to convert ...
6 years, 4 months ago (2014-08-04 17:52:09 UTC) #1
pavely
lgtm
6 years, 4 months ago (2014-08-04 18:31:37 UTC) #2
brettw
lgtm
6 years, 4 months ago (2014-08-04 18:33:39 UTC) #3
rlarocque
The CQ bit was checked by rlarocque@chromium.org
6 years, 4 months ago (2014-08-04 18:37:48 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rlarocque@chromium.org/439533004/1
6 years, 4 months ago (2014-08-04 18:39:20 UTC) #5
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: chromium_presubmit on tryserver.chromium.linux ...
6 years, 4 months ago (2014-08-04 21:09:21 UTC) #6
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 4 months ago (2014-08-04 21:13:00 UTC) #7
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/1920)
6 years, 4 months ago (2014-08-04 21:13:01 UTC) #8
rlarocque
That's odd. The presubmit scripts and the chromite butler disagree about whether or not all ...
6 years, 4 months ago (2014-08-04 21:27:01 UTC) #9
rlarocque
The CQ bit was checked by rlarocque@chromium.org
6 years, 4 months ago (2014-08-04 21:27:22 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rlarocque@chromium.org/439533004/1
6 years, 4 months ago (2014-08-04 21:28:40 UTC) #11
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 4 months ago (2014-08-05 04:14:27 UTC) #12
commit-bot: I haz the power
Failed to apply patch for components/invalidation/BUILD.gn: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 4 months ago (2014-08-05 04:14:28 UTC) #13
rlarocque
It conflicted with my own jingle GN patch. :( Fixed the conflict, rebased, and re-uploaded.
6 years, 4 months ago (2014-08-05 17:50:07 UTC) #14
rlarocque
The CQ bit was checked by rlarocque@chromium.org
6 years, 4 months ago (2014-08-05 17:50:12 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rlarocque@chromium.org/439533004/20001
6 years, 4 months ago (2014-08-05 17:51:31 UTC) #16
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux ...
6 years, 4 months ago (2014-08-05 21:43:51 UTC) #17
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 4 months ago (2014-08-05 23:08:14 UTC) #18
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_swarming/builds/1383)
6 years, 4 months ago (2014-08-05 23:08:15 UTC) #19
rlarocque
The CQ bit was checked by rlarocque@chromium.org
6 years, 4 months ago (2014-08-06 19:58:48 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rlarocque@chromium.org/439533004/40001
6 years, 4 months ago (2014-08-06 20:00:40 UTC) #21
nyquist
lgtm
6 years, 4 months ago (2014-08-06 20:05:32 UTC) #22
commit-bot: I haz the power
6 years, 4 months ago (2014-08-07 00:05:25 UTC) #23
Message was sent while issue was closed.
Change committed as 287905

Powered by Google App Engine
This is Rietveld 408576698