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

Issue 418573002: Don't add implicit deps to copy commands in GN. (Closed)

Created:
6 years, 5 months ago by brettw
Modified:
6 years, 5 months ago
Reviewers:
scottmg
CC:
chromium-reviews, tfarina
Project:
chromium
Visibility:
Public.

Description

Don't add implicit deps to copy commands in GN. As explained in the long comment in ninja_copy_target_writer.cc, writing implicit deps for the copy command is both unnecessary and causes problems given the way Chrome specifies the command. This also updates the expectations for the binary output given the new variables added in a previous patch (this test isn't being run on the bot yet). R=scottmg@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285052

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -18 lines) Patch
M tools/gn/function_toolchain.cc View 2 chunks +13 lines, -1 line 0 comments Download
M tools/gn/ninja_binary_target_writer_unittest.cc View 5 chunks +15 lines, -0 lines 0 comments Download
M tools/gn/ninja_copy_target_writer.cc View 2 chunks +22 lines, -4 lines 0 comments Download
M tools/gn/ninja_copy_target_writer_unittest.cc View 3 chunks +2 lines, -13 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
brettw
6 years, 5 months ago (2014-07-23 19:41:38 UTC) #1
scottmg
lgtm
6 years, 5 months ago (2014-07-23 19:45:21 UTC) #2
brettw
6 years, 5 months ago (2014-07-23 22:05:27 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r285052 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698