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

Unified Diff: tools/telemetry/telemetry/core/util.py

Issue 321773002: [PowerProfiler] Make sure correct version of pySerial is imported with monsoon profiler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 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: tools/telemetry/telemetry/core/util.py
diff --git a/tools/telemetry/telemetry/core/util.py b/tools/telemetry/telemetry/core/util.py
index efeab5aa7f8eaf6af6e20741002397a34ea5411d..e5114693686f8534025a9fa45077c9940ddf52fb 100644
--- a/tools/telemetry/telemetry/core/util.py
+++ b/tools/telemetry/telemetry/core/util.py
@@ -38,8 +38,10 @@ def GetChromiumSrcDir():
def AddDirToPythonPath(*path_parts):
path = os.path.abspath(os.path.join(*path_parts))
- if os.path.isdir(path) and path not in sys.path:
- sys.path.append(path)
+ if os.path.isdir(path):
+ if path in sys.path:
+ sys.path.remove(path)
qsr 2014/06/11 12:24:21 Do we really want to reinsert if it is already in
vivekg 2014/06/11 12:41:41 The reason for removing the existing path is to as
qsr 2014/06/11 13:20:54 What is the use case where it would be in the sys
+ sys.path.insert(0, path)
_counter = [0]
def _GetUniqueModuleName():
« no previous file with comments | « no previous file | tools/telemetry/telemetry/core/util_unittest.py » ('j') | tools/telemetry/telemetry/core/util_unittest.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698