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

Issue 23533016: Revert 220846 "Make landmines.py take an extra script via an env..." (Closed)

Created:
7 years, 3 months ago by tkent
Modified:
7 years, 3 months ago
Reviewers:
Siva Chandra
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 220846 "Make landmines.py take an extra script via an env..." It broke telemetry_unittests. http://build.chromium.org/p/chromium.win/builders/Win%207%20Tests%20x64%20%283%29/builds/8782/steps/telemetry_unittests/logs/stdio E:\b\depot_tools\python_bin\python_slave.exe src\tools\telemetry\run_tests --browser=release_x64 No adb found in $PATH, fallback to checked in binary. Traceback (most recent call last): File "src\tools\telemetry\run_tests", line 8, in <module> from telemetry.unittest import gtest_testrunner File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\__init__.py", line 12, in <module> from telemetry.core.browser_options import BrowserFinderOptions File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\browser_options.py", line 12, in <module> from telemetry.core import browser_finder File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\browser_finder.py", line 9, in <module> from telemetry.core.backends.chrome import android_browser_finder File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\backends\chrome\android_browser_finder.py", line 17, in <module> from telemetry.core.backends.chrome import android_browser_backend File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\backends\chrome\android_browser_backend.py", line 15, in <module> from telemetry.core.backends.chrome import chrome_browser_backend File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\backends\chrome\chrome_browser_backend.py", line 19, in <module> from telemetry.core.backends.chrome import extension_dict_backend File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\backends\chrome\extension_dict_backend.py", line 9, in <module> from telemetry.core.backends.chrome import inspector_backend File "E:\b\build\slave\Win_7_Tests_x64__3_\build\src\tools\telemetry\telemetry\core\backends\chrome\inspector_backend.py", line 19, in <module> from telemetry.core.jsheap import model ImportError: No module named jsheap > Make landmines.py take an extra script via an env var. > > The extra Python script to run can be specified via env var > EXTRA_LANDMINES_SCRIPT. > > BUG=223636 > > Review URL: https://chromiumcodereview.appspot.com/23604016 TBR=sivachandra@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220853

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -14 lines) Patch
M trunk/src/build/landmines.py View 2 chunks +3 lines, -14 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tkent
7 years, 3 months ago (2013-09-02 21:04:17 UTC) #1
tkent
7 years, 3 months ago (2013-09-02 21:04:28 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r220853.

Powered by Google App Engine
This is Rietveld 408576698