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

Unified Diff: build/android/pylib/constants.py

Issue 22903016: [android] Adds constants.GetOutDirectory() and converts test scripts to use it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update adb_install_apk.py Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/android/pylib/constants.py
diff --git a/build/android/pylib/constants.py b/build/android/pylib/constants.py
index bf35fff6e7b7c9f9b454f2972301d244ddcbf2ff..6a770f16fc1cc43aa33679a6dda9c13f13fa95d2 100644
--- a/build/android/pylib/constants.py
+++ b/build/android/pylib/constants.py
@@ -94,6 +94,18 @@ def SetBuildType(build_type):
os.environ['BUILDTYPE'] = build_type
+def GetBuildDirectory(build_type=None):
bulach 2013/09/02 09:11:18 hmm... "BuildDirectory" is a bit ambiguous, there'
craigdh 2013/09/03 18:36:09 Done.
+ """Returns the build directory.
+
+ Args:
+ build_type: Build type, generally 'Debug' or 'Release'. Defaults to the
+ globally set build type environment variable BUILDTYPE.
+ """
+ return os.path.abspath(os.path.join(
+ DIR_SOURCE_ROOT, os.environ.get('CHROMIUM_OUT_DIR', 'out'),
frankf 2013/08/27 00:44:57 who sets this?
craigdh 2013/08/27 16:57:17 I'm not sure who overrides the out directory (took
bulach 2013/09/02 09:11:18 yeah, the idea behind this is that we'd eventually
+ GetBuildType() if build_type is None else build_type))
+
+
def _GetADBPath():
if os.environ.get('ANDROID_SDK_ROOT'):
return 'adb'

Powered by Google App Engine
This is Rietveld 408576698