| Index: scripts/slave/bot_update.py
|
| diff --git a/scripts/slave/bot_update.py b/scripts/slave/bot_update.py
|
| index 1a2e5974c9b4f7ffd237ee1fb8032cf15b71151d..c5db40b27ed683b43905da57cf7fc08e9cb018bf 100755
|
| --- a/scripts/slave/bot_update.py
|
| +++ b/scripts/slave/bot_update.py
|
| @@ -42,24 +42,8 @@
|
| SCRIPTS_DIR = path.dirname(THIS_DIR)
|
| BUILD_DIR = path.dirname(SCRIPTS_DIR)
|
| ROOT_DIR = path.dirname(BUILD_DIR)
|
| +BUILD_INTERNAL_DIR = path.join(ROOT_DIR, 'build_internal')
|
| DEPOT_TOOLS_DIR = path.join(ROOT_DIR, 'depot_tools')
|
| -
|
| -# TODO(luqui): This is a horrible hack to identify build_internal when build
|
| -# is a recipe dependency. bot_update should not be depending on internal,
|
| -# rather the arrow should go the other way (or just be destroyed).
|
| -def check_build_internal(d):
|
| - d = path.abspath(d)
|
| - if path.basename(d) == 'build_internal' and path.isdir(d):
|
| - return d
|
| - else:
|
| - return None
|
| -
|
| -BUILD_INTERNAL_DIR = (
|
| - check_build_internal(path.join(ROOT_DIR, 'build_internal')) or
|
| - check_build_internal(path.join(ROOT_DIR, # .recipe_deps
|
| - path.pardir, # slave
|
| - path.pardir, # scripts
|
| - path.pardir))) # build_internal
|
|
|
|
|
| CHROMIUM_GIT_HOST = 'https://chromium.googlesource.com'
|
| @@ -203,7 +187,7 @@
|
|
|
|
|
| internal_data = {}
|
| -if BUILD_INTERNAL_DIR:
|
| +if os.path.isdir(BUILD_INTERNAL_DIR):
|
| local_vars = {}
|
| try:
|
| execfile(os.path.join(
|
| @@ -332,12 +316,7 @@
|
|
|
| # Find the patch tool.
|
| if sys.platform.startswith('win'):
|
| - if not BUILD_INTERNAL_DIR:
|
| - print 'Warning: could not find patch tool because there is no '
|
| - print 'build_internal present.'
|
| - PATCH_TOOL = None
|
| - else:
|
| - PATCH_TOOL = path.join(BUILD_INTERNAL_DIR, 'tools', 'patch.EXE')
|
| + PATCH_TOOL = path.join(BUILD_INTERNAL_DIR, 'tools', 'patch.EXE')
|
| else:
|
| PATCH_TOOL = '/usr/bin/patch'
|
|
|
|
|