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

Issue 133963003: android: Stop adding toolchain dir to PATH. (Closed)

Created:
6 years, 11 months ago by Nico
Modified:
6 years, 11 months ago
Reviewers:
Torne
CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

android: Stop adding toolchain dir to PATH. This seems mostly no longer necessary (at least I can build all targets in an open-source chrome/android build without problems). I'm guessing that https://codereview.chromium.org/9693042 made this obsolete. It was only needed for `as` with clang; pass -B to clang instead for now (until we switch to clang's integrated assembler). BUG=330631, 124610 R=torne@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246714

Patch Set 1 #

Patch Set 2 : rebase #

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

Messages

Total messages: 5 (0 generated)
Nico
6 years, 11 months ago (2014-01-23 00:32:55 UTC) #1
Nico
ping
6 years, 11 months ago (2014-01-23 17:01:46 UTC) #2
Torne
It was semi-useful in the past for manually invoking arm-linux-androideabi-{objdump,nm,readelf} and friends, but most people ...
6 years, 11 months ago (2014-01-23 21:48:57 UTC) #3
Nico
Thanks! On 2014/01/23 21:48:57, Torne wrote: > It was semi-useful in the past for manually ...
6 years, 11 months ago (2014-01-23 23:14:19 UTC) #4
Nico
6 years, 11 months ago (2014-01-23 23:17:21 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r246714 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698