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

Side by Side Diff: tools/find_depot_tools.py

Issue 7677019: Fix infinite recursion in find_depot_tools (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 """Small utility function to find depot_tools and add it to the python path. 4 """Small utility function to find depot_tools and add it to the python path.
5 5
6 Will throw an ImportError exception if depot_tools can't be found since it 6 Will throw an ImportError exception if depot_tools can't be found since it
7 imports breakpad. 7 imports breakpad.
8 """ 8 """
9 9
10 import os 10 import os
11 import sys 11 import sys
12 12
13 def add_depot_tools_to_path(): 13 def add_depot_tools_to_path():
14 """Search for depot_tools and add it to sys.path.""" 14 """Search for depot_tools and add it to sys.path."""
15 # First look if depot_tools is already in PYTHONPATH. 15 # First look if depot_tools is already in PYTHONPATH.
16 for i in sys.path: 16 for i in sys.path:
17 if i.rstrip(os.sep).endswith('depot_tools'): 17 if i.rstrip(os.sep).endswith('depot_tools'):
18 return i 18 return i
19 # Then look if depot_tools is in PATH, common case. 19 # Then look if depot_tools is in PATH, common case.
20 for i in os.environ['PATH'].split(os.pathsep): 20 for i in os.environ['PATH'].split(os.pathsep):
21 if i.rstrip(os.sep).endswith('depot_tools'): 21 if i.rstrip(os.sep).endswith('depot_tools'):
22 sys.path.append(i.rstrip(os.sep)) 22 sys.path.append(i.rstrip(os.sep))
23 return i 23 return i
24 # Rare case, it's not even in PATH, look upward up to root. 24 # Rare case, it's not even in PATH, look upward up to root.
25 root_dir = os.path.dirname(os.path.abspath(__file__)) 25 root_dir = os.path.dirname(os.path.abspath(__file__))
26 while root_dir: 26 previous_dir = os.path.abspath(__file__)
27 while root_dir and root_dir != previous_dir:
27 if os.path.isfile(os.path.join(root_dir, 'depot_tools', 'breakpad.py')): 28 if os.path.isfile(os.path.join(root_dir, 'depot_tools', 'breakpad.py')):
28 i = os.path.join(root_dir, 'depot_tools') 29 i = os.path.join(root_dir, 'depot_tools')
29 sys.path.append(i) 30 sys.path.append(i)
30 return i 31 return i
32 previous_dir = root_dir
31 root_dir = os.path.dirname(root_dir) 33 root_dir = os.path.dirname(root_dir)
32 print >> sys.stderr, 'Failed to find depot_tools' 34 print >> sys.stderr, 'Failed to find depot_tools'
33 return None 35 return None
34 36
35 add_depot_tools_to_path() 37 add_depot_tools_to_path()
36 38
37 # pylint: disable=W0611 39 # pylint: disable=W0611
38 import breakpad 40 import breakpad
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698