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

Issue 132793008: Revert 248878 "Merge 248802 "Remove build references to third_pa..." (Closed)

Created:
6 years, 10 months ago by dxie
Modified:
6 years, 10 months ago
Reviewers:
dxie
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Revert 248878 "Merge 248802 "Remove build references to third_pa..." > Merge 248802 "Remove build references to third_party/cygwin" > > > Remove build references to third_party/cygwin > > > > There's test running dependencies in blink and buildbot, > > so the directory/DEPS can't be removed right now. For the > > time being, remove the build related entries. > > > > R=maruel@chromium.org, thakis@chromium.org > > BUG=123026 > > TEST=remove from DEPS and .DEPS.git, delete HKCU\Software\Cygnus Solutions\... and build successfully. > > > > Review URL: https://codereview.chromium.org/149063008 > > TBR=scottmg@chromium.org > > Review URL: https://codereview.chromium.org/153393003 TBR=dxie@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248879

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, --3 lines) Patch
M DEPS View 1 chunk +6 lines, -0 lines 0 comments Download
M build/common.gypi View 1 chunk +1 line, -0 lines 0 comments Download
A + build/win/setup_cygwin_mount.py View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/browser_tests.isolate View 1 chunk +2 lines, -0 lines 0 comments Download
A + chrome/tools/build/win/version.bat View 0 chunks +-1 lines, --1 lines 0 comments Download
A + chrome/tools/build/win/version.rules View 0 chunks +-1 lines, --1 lines 0 comments Download
M content/content_browsertests.isolate View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
dxie
6 years, 10 months ago (2014-02-05 06:37:03 UTC) #1
dxie
6 years, 10 months ago (2014-02-05 06:37:13 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r248879.

Powered by Google App Engine
This is Rietveld 408576698