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

Issue 1406323002: Revert "add gflags to deps to allow command line parameters." (Closed)

Created:
5 years, 2 months ago by kjellander_chromium
Modified:
5 years, 1 month ago
Reviewers:
fbarchard
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

Revert "add gflags to deps to allow command line parameters." This reverts commit 2dd3d9230ee663e71ed4ad9164033ed672e571de. Reason: chromium_git is a missing variable, and to properly add gflags, we need to check in GYP files in third_party/gflags first, then add the DEPS entry. BUG=libyuv:507 TBR=fbarchard@chromium.org Committed: https://chromium.googlesource.com/libyuv/libyuv/+/f80cc26da78b8221c235875e5954b5e647426640

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -12 lines) Patch
M DEPS View 1 chunk +0 lines, -10 lines 0 comments Download
M README.chromium View 1 chunk +1 line, -1 line 0 comments Download
M include/libyuv/version.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (1 generated)
kjellander_chromium
5 years, 2 months ago (2015-10-16 19:46:53 UTC) #2
kjellander_chromium
Committed patchset #1 (id:1) manually as f80cc26da78b8221c235875e5954b5e647426640 (presubmit successful).
5 years, 2 months ago (2015-10-16 19:47:04 UTC) #3
fbarchard
5 years, 1 month ago (2015-10-26 23:11:32 UTC) #4
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698