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

Issue 65083002: Remove extra quoting from the Linux GN platform filter. (Closed)

Created:
7 years, 1 month ago by brettw
Modified:
7 years, 1 month ago
Reviewers:
Isaac (away), shadi
CC:
chromium-reviews
Visibility:
Public.

Description

Remove extra quoting from the Linux GN platform filter. This expression is not parsed by the shell, so is not subject to expansion, so it doesn't need to be quoted. And the quotes were making the regular expression not match. R=ilevy@chromium.org, shadi@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233683

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
brettw
7 years, 1 month ago (2013-11-07 19:16:25 UTC) #1
Isaac (away)
lgtm (drive by)
7 years, 1 month ago (2013-11-07 19:29:57 UTC) #2
shadi
LGTM Thanks!
7 years, 1 month ago (2013-11-07 20:37:15 UTC) #3
brettw
7 years, 1 month ago (2013-11-07 21:08:47 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r233683 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698