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

Issue 105123002: Revert 238718 "Fix non-goma android build after r238630" (Closed)

Created:
7 years ago by boliu
Modified:
7 years ago
Reviewers:
boliu
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 238718 "Fix non-goma android build after r238630" Broke gyp on chromeos bot (possibly others) > Fix non-goma android build after r238630 > > dirname outputs "." when given an empty argument, which gyp rules then > interpret as goma being available. > > Remove ANDROID_GOMA_WRAPPER which is no longer used, and use GOMA_DIR > directly for android build. > > BUG= > NOTRY=true > > Review URL: https://codereview.chromium.org/104563004 TBR=boliu@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238719

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M trunk/src/build/android/envsetup.sh View 2 chunks +3 lines, -0 lines 0 comments Download
M trunk/src/build/common.gypi View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
boliu
7 years ago (2013-12-04 17:52:07 UTC) #1
boliu
7 years ago (2013-12-04 17:52:18 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r238719.

Powered by Google App Engine
This is Rietveld 408576698