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

Side by Side Diff: scripts/slave/recipe_modules/ios/api.py

Issue 2146873002: ios: drop dependency on chromium_tests; move analyze to filter module (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 5 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
« no previous file with comments | « scripts/slave/recipe_modules/ios/__init__.py ('k') | scripts/slave/recipes/chromium_trybot.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 4
5 import copy 5 import copy
6 6
7 from recipe_engine import recipe_api 7 from recipe_engine import recipe_api
8 8
9 9
10 class iOSApi(recipe_api.RecipeApi): 10 class iOSApi(recipe_api.RecipeApi):
(...skipping 305 matching lines...) Expand 10 before | Expand all | Expand 10 after
316 if (use_analyze and 316 if (use_analyze and
317 self.compiler == 'ninja' and 317 self.compiler == 'ninja' and
318 self.m.tryserver.is_tryserver and 318 self.m.tryserver.is_tryserver and
319 'without patch' not in suffix): 319 'without patch' not in suffix):
320 affected_files = self.m.tryserver.get_files_affected_by_patch() 320 affected_files = self.m.tryserver.get_files_affected_by_patch()
321 # The same test may be configured to run on multiple simulators. 321 # The same test may be configured to run on multiple simulators.
322 # Only specify each test once for the analyzer. 322 # Only specify each test once for the analyzer.
323 tests = list(set(test['app'] for test in self.__config['tests'])) 323 tests = list(set(test['app'] for test in self.__config['tests']))
324 324
325 test_targets, compile_targets = ( 325 test_targets, compile_targets = (
326 self.m.chromium_tests.analyze( 326 self.m.filter.analyze(
327 affected_files, 327 affected_files,
328 tests, 328 tests,
329 self.__config['additional_compile_targets'] + tests, 329 self.__config['additional_compile_targets'] + tests,
330 'trybot_analyze_config.json', 330 'trybot_analyze_config.json',
331 additional_names=['chromium', 'ios'], 331 additional_names=['chromium', 'ios'],
332 mb_mastername=self.__config['mastername'], 332 mb_mastername=self.__config['mastername'],
333 ) 333 )
334 ) 334 )
335 335
336 test_targets = set(test_targets) 336 test_targets = set(test_targets)
(...skipping 161 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 self.configuration, 498 self.configuration,
499 'iossim', 499 'iossim',
500 ), 500 ),
501 'ninja': self.m.path.join( 501 'ninja': self.m.path.join(
502 'src', 502 'src',
503 build_dir, 503 build_dir,
504 '%s-%s' % (self.configuration, platform), 504 '%s-%s' % (self.configuration, platform),
505 'iossim', 505 'iossim',
506 ), 506 ),
507 }[self.compiler] 507 }[self.compiler]
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/ios/__init__.py ('k') | scripts/slave/recipes/chromium_trybot.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698