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

Side by Side Diff: infra/bots/recipe_modules/vars/api.py

Issue 2209343002: [recipes] Stop pretending that buildbot_spec is a step, simplify (Closed) Base URL: https://skia.googlesource.com/skia.git@merge_buildbot_spec_sub1
Patch Set: rebase 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
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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 5
6 # pylint: disable=W0201 6 # pylint: disable=W0201
7 7
8 8
9 from recipe_engine import recipe_api 9 from recipe_engine import recipe_api
10 import os 10 import os
(...skipping 765 matching lines...) Expand 10 before | Expand all | Expand 10 after
776 'SAN' in self.builder_name) or 776 'SAN' in self.builder_name) or
777 'RecreateSKPs' in self.builder_name): 777 'RecreateSKPs' in self.builder_name):
778 self.need_chromium_checkout = True 778 self.need_chromium_checkout = True
779 if 'RecreateSKPs' in self.builder_name: 779 if 'RecreateSKPs' in self.builder_name:
780 self.gclient_env['CPPFLAGS'] = ( 780 self.gclient_env['CPPFLAGS'] = (
781 '-DSK_ALLOW_CROSSPROCESS_PICTUREIMAGEFILTERS=1') 781 '-DSK_ALLOW_CROSSPROCESS_PICTUREIMAGEFILTERS=1')
782 782
783 # Some bots also require a checkout of PDFium. 783 # Some bots also require a checkout of PDFium.
784 self.need_pdfium_checkout = 'PDFium' in self.builder_name 784 self.need_pdfium_checkout = 'PDFium' in self.builder_name
785 785
786 786 # Obtain the spec for this builder. Use it to set more properties.
787 def update_with_builder_spec(self, builder_spec): 787 self.builder_spec = get_builder_spec(self.m, self.builder_name)
788 """Set more variables based on the builder_spec."""
789 # Obtain the spec for this builder from the Skia repo. Use it to set more
790 # properties.
791 self.builder_spec = builder_spec
792 788
793 self.builder_cfg = self.builder_spec['builder_cfg'] 789 self.builder_cfg = self.builder_spec['builder_cfg']
794 self.role = self.builder_cfg['role'] 790 self.role = self.builder_cfg['role']
795 791
796 self.configuration = self.builder_spec['configuration'] 792 self.configuration = self.builder_spec['configuration']
797 self.default_env.update({'SKIA_OUT': self.skia_out, 793 self.default_env.update({'SKIA_OUT': self.skia_out,
798 'BUILDTYPE': self.configuration}) 794 'BUILDTYPE': self.configuration})
799 self.default_env.update(self.builder_spec['env']) 795 self.default_env.update(self.builder_spec['env'])
800 self.build_targets = [str(t) for t in self.builder_spec['build_targets']] 796 self.build_targets = [str(t) for t in self.builder_spec['build_targets']]
801 self.do_compile_steps = self.builder_spec.get('do_compile_steps', True) 797 self.do_compile_steps = self.builder_spec.get('do_compile_steps', True)
802 self.do_test_steps = self.builder_spec['do_test_steps'] 798 self.do_test_steps = self.builder_spec['do_test_steps']
803 self.do_perf_steps = self.builder_spec['do_perf_steps'] 799 self.do_perf_steps = self.builder_spec['do_perf_steps']
804 self.is_trybot = self.builder_cfg['is_trybot'] 800 self.is_trybot = self.builder_cfg['is_trybot']
805 self.issue = None 801 self.issue = None
806 self.patchset = None 802 self.patchset = None
807 self.rietveld = None 803 self.rietveld = None
808 if self.is_trybot: 804 if self.is_trybot:
809 self.issue = self.m.properties['issue'] 805 self.issue = self.m.properties['issue']
810 self.patchset = self.m.properties['patchset'] 806 self.patchset = self.m.properties['patchset']
811 self.rietveld = self.m.properties['rietveld'] 807 self.rietveld = self.m.properties['rietveld']
812 self.upload_dm_results = self.builder_spec['upload_dm_results'] 808 self.upload_dm_results = self.builder_spec['upload_dm_results']
813 self.upload_perf_results = self.builder_spec['upload_perf_results'] 809 self.upload_perf_results = self.builder_spec['upload_perf_results']
814 self.dm_dir = self.m.path.join( 810 self.dm_dir = self.m.path.join(
815 self.swarming_out_dir, 'dm') 811 self.swarming_out_dir, 'dm')
816 self.perf_data_dir = self.m.path.join(self.swarming_out_dir, 812 self.perf_data_dir = self.m.path.join(self.swarming_out_dir,
817 'perfdata', self.builder_name, 'data') 813 'perfdata', self.builder_name, 'data')
818 self.dm_flags = self.builder_spec['dm_flags'] 814 self.dm_flags = self.builder_spec['dm_flags']
819 self.nanobench_flags = self.builder_spec['nanobench_flags'] 815 self.nanobench_flags = self.builder_spec['nanobench_flags']
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698