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

Unified Diff: chrome/test/nacl_test_injection/find_chrome.py

Issue 26718002: Remove references to sconsbuild (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 2 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: chrome/test/nacl_test_injection/find_chrome.py
diff --git a/chrome/test/nacl_test_injection/find_chrome.py b/chrome/test/nacl_test_injection/find_chrome.py
index 186a83b2e6a22a3facbe80fefba74aa43a9f824f..e6d2b24c4adaa076d0c9c7d948c3abb7b29abb36 100755
--- a/chrome/test/nacl_test_injection/find_chrome.py
+++ b/chrome/test/nacl_test_injection/find_chrome.py
@@ -25,13 +25,11 @@ def FindChrome(src_dir, configs, verbose=False):
chrome_locations.extend([
'build/%s/chrome.exe' % config,
'chrome/%s/chrome.exe' % config,
- # For Linux buildbots. scripts/slave/extract_build.py extracts builds
- # to src/sconsbuild/ rather than src/out/.
Nico 2013/10/09 17:36:40 This is not true, see any Linux tester output (e.g
Lei Zhang 2013/10/09 18:59:14 I think it was true at one point. Probably not any
- 'sconsbuild/%s/chrome' % config,
# Windows Chromium ninja builder
'out/%s/chrome.exe' % config,
+ # Linux
'out/%s/chrome' % config,
- # Mac Chromium make builder
+ # Mac Chromium ninja builder
'out/%s/Chromium.app/Contents/MacOS/Chromium' % config,
# Mac release make builder
'out/%s/Google Chrome.app/Contents/MacOS/Google Chrome' % config,

Powered by Google App Engine
This is Rietveld 408576698