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

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

Issue 2187253003: Fix applying non-chromium patches to ios trybots (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Fix Created 4 years, 4 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 | Annotate | Revision Log
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 319 matching lines...) Expand 10 before | Expand all | Expand 10 after
330 self.m.properties['buildername'], 330 self.m.properties['buildername'],
331 name='generate_build_files' + suffix, 331 name='generate_build_files' + suffix,
332 mb_config_path=mb_config_path, 332 mb_config_path=mb_config_path,
333 build_dir='//out/' + build_sub_path) 333 build_dir='//out/' + build_sub_path)
334 334
335 use_analyze = self.__config['use_analyze'] 335 use_analyze = self.__config['use_analyze']
336 if (use_analyze and 336 if (use_analyze and
337 self.compiler == 'ninja' and 337 self.compiler == 'ninja' and
338 self.m.tryserver.is_tryserver and 338 self.m.tryserver.is_tryserver and
339 'without patch' not in suffix): 339 'without patch' not in suffix):
340 affected_files = self.m.tryserver.get_files_affected_by_patch() 340 affected_files = self.m.chromium_tests.get_files_affected_by_patch()
341 # The same test may be configured to run on multiple simulators. 341 # The same test may be configured to run on multiple simulators.
342 # Only specify each test once for the analyzer. 342 # Only specify each test once for the analyzer.
343 tests = list(set(test['app'] for test in self.__config['tests'])) 343 tests = list(set(test['app'] for test in self.__config['tests']))
344 344
345 test_targets, compile_targets = ( 345 test_targets, compile_targets = (
346 self.m.filter.analyze( 346 self.m.filter.analyze(
347 affected_files, 347 affected_files,
348 tests, 348 tests,
349 self.__config['additional_compile_targets'] + tests, 349 self.__config['additional_compile_targets'] + tests,
350 'trybot_analyze_config.json', 350 'trybot_analyze_config.json',
(...skipping 392 matching lines...) Expand 10 before | Expand all | Expand 10 after
743 self.configuration, 743 self.configuration,
744 'iossim', 744 'iossim',
745 ), 745 ),
746 'ninja': self.m.path.join( 746 'ninja': self.m.path.join(
747 'src', 747 'src',
748 build_dir, 748 build_dir,
749 '%s-%s' % (self.configuration, platform), 749 '%s-%s' % (self.configuration, platform),
750 'iossim', 750 'iossim',
751 ), 751 ),
752 }[self.compiler] 752 }[self.compiler]
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipes/ios/try.py » ('j') | scripts/slave/recipes/ios/try.expected/icu_patch.json » ('J')

Powered by Google App Engine
This is Rietveld 408576698