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

Unified Diff: appengine/findit/waterfall/extractor.py

Issue 838003004: [Findit] Add three sub-pipelines to analyze build failure. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Address comments. Created 5 years, 11 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: appengine/findit/waterfall/extractor.py
diff --git a/appengine/findit/waterfall/extractor.py b/appengine/findit/waterfall/extractor.py
index 1c30acfb4894aa3f7d41ed8ee50cf0d697442168..81de3d56b4b2c17e11a65ba19ec5a06ec46626ff 100644
--- a/appengine/findit/waterfall/extractor.py
+++ b/appengine/findit/waterfall/extractor.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2014 The Chromium Authors. All rights reserved.
+# Copyright 2014 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
@@ -13,7 +13,9 @@ class Extractor(object):
match = extractor_util.PYTHON_STACK_TRACE_PATTERN.match(message_line)
if match:
trace_line = match.groupdict()
- failure_signal.AddFile(trace_line['file'], trace_line['line'])
+ failure_signal.AddFile(
+ extractor_util.NormalizeFilePath(trace_line['file']),
+ trace_line['line'])
else:
for match in extractor_util.FILE_PATH_LINE_PATTERN.finditer(message_line):
file_path, line_number = match.groups()

Powered by Google App Engine
This is Rietveld 408576698